Skip to content

remove system file when testing#4

Open
glensc wants to merge 3 commits into
masterfrom
3-sys-file
Open

remove system file when testing#4
glensc wants to merge 3 commits into
masterfrom
3-sys-file

Conversation

@glensc

@glensc glensc commented Sep 8, 2017

Copy link
Copy Markdown
Member

solves #3 using solution "1"

@glensc

glensc commented Sep 8, 2017

Copy link
Copy Markdown
Member Author

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

glensc commented Sep 26, 2017

Copy link
Copy Markdown
Member Author

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

thomasjfox commented Sep 26, 2017 via email

Copy link
Copy Markdown
Contributor

@glensc

glensc commented Sep 26, 2017

Copy link
Copy Markdown
Member Author

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

thomasjfox commented Sep 27, 2017 via email

Copy link
Copy Markdown
Contributor

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.

3 participants