Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove system file when testing #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

remove system file when testing #4

wants to merge 3 commits into from

Conversation

glensc
Copy link
Member

@glensc glensc commented Sep 8, 2017

solves #3 using solution "1"

@glensc
Copy link
Member Author

glensc commented Sep 8, 2017

can't test osx part because travis macos infrastructure is down:
https://www.traviscistatus.com/
https://www.traviscistatus.com/incidents/4qylrqvy50gy

unable to replace files in /usr on osx
@glensc
Copy link
Member Author

glensc commented Sep 26, 2017

https://travis-ci.org/file/file-tests/jobs/279692965

@thomasjfox do you know how the test data is modified?

on OSX an x.iso is detected differently:

FAIL db/iso/x.iso
Expected :DOS/MBR boot sector; partition 1 : ID=0x17, active, start-CHS (0x0,0,1), end-CHS (0x2c7,63,32), startsector 0, 1458176 sectors
Got      :ISO 9660 CD-ROM filesystem data (DOS/MBR boot sector) 'testday-20110324' (bootable)
Expected :application/octet-stream; charset=binary
Got      :application/x-iso9660-image; charset=binary

https://travis-ci.org/file/file-tests/jobs/279692965#L643

@thomasjfox
Copy link
Contributor

thomasjfox commented Sep 26, 2017 via email

@glensc
Copy link
Member Author

glensc commented Sep 26, 2017

but linux build succeeds on the same build:

and i don't understand how the "diff" is made, as i read from readme, the .pickle files should be created from previous file release, but travis job currently present doesn't do that. it just runs diff of current codebase. i would suspect first cache from previous runs, but there's nothing configured to be cached between travis builds...

@thomasjfox
Copy link
Contributor

thomasjfox commented Sep 27, 2017 via email

This was referenced Jan 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants