Skip to content

Add system_provided to extra_requires#40

Merged
AyanSinhaMahapatra merged 8 commits intomainfrom
system-provided
Mar 23, 2026
Merged

Add system_provided to extra_requires#40
AyanSinhaMahapatra merged 8 commits intomainfrom
system-provided

Conversation

@JonoYang
Copy link
Member

* Update tests

@JonoYang
Copy link
Member Author

@pombredanne We have about ~8 tests that have different filetype or mimetypes between the linux and macos test runs: 1e3c70c

Do you have any ideas on how to normalize the difference in filetype/mimetype? Should we keep both results and check against them?

@AyanSinhaMahapatra
Copy link
Member

Should we keep both results and check against them?

@JonoYang I think we should just do this, also encountered this in a different branch

@AyanSinhaMahapatra AyanSinhaMahapatra force-pushed the system-provided branch 7 times, most recently from 035e9dd to 6b47191 Compare March 23, 2026 13:13
Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
@AyanSinhaMahapatra AyanSinhaMahapatra force-pushed the system-provided branch 5 times, most recently from eb924a7 to a20795a Compare March 23, 2026 14:39
Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JonoYang Thanks++
I've added the mac/linux libmagic system provided test expectations as seperate files to make the tests pass on macos arm CI, and linux arm CI
I've also tried different versions of ubuntu arm, these are all supported, but there are little differences in type results so I've added for only one version.
These are added as seperate test expectation files and the ~ in the filename is not ideal, we would want to update commoncode and the way we handle these tests later.

@AyanSinhaMahapatra AyanSinhaMahapatra merged commit 335d0cf into main Mar 23, 2026
23 checks passed
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