-
Notifications
You must be signed in to change notification settings - Fork 407
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
zig: Add cflags_extra option #607
Conversation
It does not work for 2 options. E.g. |
346209b
to
383b2ac
Compare
I was afraid of that it would require separating them... I'll have a look! |
@ivmai hmm, actually, are you sure?
both
|
Please check it on gctest.c. (I mean |
This passes through extra options to the compiler. No validation is performed, the values are split on space and passed in as separate flags.
383b2ac
to
3f51614
Compare
Ok, I see what you mean @ivmai. I am getting the same result, yet the --verbose output does say the flags are passed. I guess the way it prints them to me and the way they are actually paseed somehow differs. |
@ivmai simply splitting it up in parts and appending each to the flags works. I just pushed an update. |
PR #607 (bdwgc). This passes through extra options (flags) to the C compiler. No flags validation is performed; the values given in CFLAGS_EXTRA are split on a space and append to compiler flags one by one. * build.zig (cflags_extra): New option (an empty string by default). * build.zig (flags): Append space-separated flags from cflags_extra (after all other flags are appended); remove relevant TODO item.
Merged as 70c9e87 |
This passes through extra options to the compiler. No validation is performed, the value is passed through as-is.
Part of #602