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

Do not stop compilation on compiler warnings #4338

Merged
merged 1 commit into from
Jul 14, 2024
Merged

Conversation

anatol
Copy link
Contributor

@anatol anatol commented Jul 13, 2024

Compilers like GCC keep adding new checks that produce new warnings. Sometimes it can be false positives.

Do not treat such warnings in binaryen library as errors. tinygo won't be able to provide zero warnings in its dependencies.

Closes #4332

Compilers like GCC keep adding new checks that produce new warnings.
Sometimes it can be false positives.

Do not treat such warnings in binaryen library as errors. tinygo won't
be able to provide zero warnings in its dependencies.

Closes tinygo-org#4332
@aykevl aykevl changed the base branch from release to dev July 14, 2024 12:30
@aykevl aykevl merged commit fb91c70 into tinygo-org:dev Jul 14, 2024
17 checks passed
@aykevl
Copy link
Member

aykevl commented Jul 14, 2024

Thank you!

(Some context: I believe -Werror should be on during development to catch problems, but should be disabled for release builds because compilers change and warnings can be false-positives at times).

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.

tinygo fails to build at Arch Linux clean environment
2 participants