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

Fix build on MacOS #2

Closed
wants to merge 1 commit into from
Closed

Fix build on MacOS #2

wants to merge 1 commit into from

Conversation

michaelshmitty
Copy link

@michaelshmitty michaelshmitty commented Aug 16, 2023

This changes some config flags dynamically based on the target os. It fixes some build errors on MacOS regarding the inclusion of endian.h on that platform. May need to do the same for Windows platform, but haven't set up that environment yet.
Eager to learn from this, so please let me know your thoughts if you have the time. Thanks!

Related to allyourcodebase/ffmpeg#6

@@ -335,6 +335,7 @@ pub fn build(b: *std.Build) void {
"-DHAVE_CONFIG_H",
"-std=c17",
"-Wno-implicit-function-declaration",
"-Wno-int-conversion",
Copy link
Author

Choose a reason for hiding this comment

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

Clang output a whole bunch of warnings that were turned into errors and made the build fail if I didn't disable these warnings. Not a fan of having to disable warnings though, maybe there's a better way?

@michaelshmitty michaelshmitty closed this by deleting the head repository Apr 23, 2024
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.

1 participant