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
WIP RFC gnome.py cross improvements #4261
Conversation
mesonbuild/backend/ninjabackend.py
Outdated
@@ -1396,7 +1396,7 @@ def generate_dynamic_link_rules(self, outfile): | |||
rspfile_content = $ARGS {output_args} $in $LINK_ARGS {cross_args} $aliasing | |||
''' | |||
else: | |||
command_template = ' command = {executable} $ARGS {output_args} $in $LINK_ARGS {cross_args} $aliasing\n' | |||
command_template = ' command = {executable} $ARGS {cross_args} {output_args} $in $LINK_ARGS $aliasing\n' |
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.
Please don't make such changes in an aside without rational. Argument ordering in one of those places that might or might not have consequences.
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.
I can't replicate the failures we were seeing, dropped and will bring it back with more proof if I can break it again.
f22a02a
to
84fd331
Compare
Rebased, squashed, and wrote nice messages. |
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.
LGTM, thanks!
Instead of showing just the command invoked, it's useful when debugging to also show all of the arguments.
When cross-compiling the gtk-doc calls were missing the configured c_args and c_link_args.
…iling When cross-compiling we shouldn't be passing the native c_args/c_link_args to g-ir-scanner.
No description provided.