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

Consider restructuring the various cflag options #9

Open
peterebden opened this issue Oct 29, 2021 · 0 comments
Open

Consider restructuring the various cflag options #9

peterebden opened this issue Oct 29, 2021 · 0 comments

Comments

@peterebden
Copy link
Collaborator

peterebden commented Oct 29, 2021

Right now we have four; dbg/opt and c/c++. The only way to override them is in their entirety which often means you have to repeat two (or all four) of them in order to change one thing (e.g. say you use --std=c++17 globally but you have one package with some third-party code that needs --std=c++03).

I wonder if this should get broken out to different areas, e.g. release/debug specific (-O3 vs. -g3 etc), the language standard, and additional flags (mostly warnings/errors usually). We'd need two of the first (dbg/opt) and of the second (c/c++) and just one of the latter, so it doesn't actually increase the number of options much, but would make it quite a bit more flexible and less repetitive.

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

No branches or pull requests

1 participant