Skip to content

Conversation

@xeioex
Copy link
Contributor

@xeioex xeioex commented Nov 25, 2025

No description provided.

@BalkanMadman
Copy link

As far as I understand, the compiler warnings (not -Werror) are disabled when CFLAGS are set? I think warnings are generally a good thing and don't cause harm, so they should probably be always enabled (with the intent of users bugging upstream when those pop up).

I think a better approach is to have certain required CFLAGS, like -fPIC and the warnings and the default CFLAGS. The required CFLAGS should be always set (well, because they're required) and the default CFLAGS are set if the user doesn't set the CFLAGS environment variable. I'm also not sure how well it interplays with your --with-cc-opt configure option.

@BalkanMadman
Copy link

What I imagine is something like CFLAGS="${NJS_REQUIRED_CFLAGS} ${CFLAGS:-${NJS_DEFAULT_CFLAGS}}"

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