-
-
Notifications
You must be signed in to change notification settings - Fork 21.1k
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
[SCons] Add options to override AR, AR and RC flags #91972
Conversation
SConstruct
Outdated
opts.Add("CXX", "C++ compiler binary") | ||
opts.Add("CC", "C compiler binary") | ||
opts.Add("LINK", "Linker binary") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To clarify, exposing the *flags
as done in this PR should be fine to merge already, as this will work fine.
Making it possible to actually override the binary for each step reliably will require more work and an approach similar to #91743, so we can leave that to discuss there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also add options for AS, AR, RANLIB, RC, and maybe S_compiler?
Why not? I have not tested #91743, but it seems fine to me (at least the idea).
Thanks! |
Add extra options to override flags for the assembler, archive tool and resource compiler, similar to
c**flags
andlinkflags
.