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

vala: Fix global arguments for valac #954

Merged

Conversation

arteymix
Copy link
Contributor

There's two things I would like to address here:

  • add VALAFLAGS environment variable support
  • fix global arguments provided by -Dvala_args which are not considered at this point

For VALAFLAGS, it is already supported by autotools and waf, so there's not excuse not to have it as well.

I tried to dig a bit into the code to see why the arguments weren't set, but couldn't find anything so far.

@jpakkane
Copy link
Member

jpakkane commented Nov 2, 2016

⚔ conflicts.

@arteymix
Copy link
Contributor Author

arteymix commented Nov 2, 2016

@jpakkane It's not meant to be merged now, I still not have figured how to fix -Dvala_args.

@arteymix arteymix force-pushed the wip/vala-fix-external-args branch 3 times, most recently from e6de2cd to 67baeec Compare November 2, 2016 22:27
@arteymix
Copy link
Contributor Author

arteymix commented Nov 7, 2016

The only thing that is missing is passing -Dvala_args during the initial configure step with meson.py. Adding the flag after with mesonconf.py works as expected and the build considers VALAFLAGS as well.

@arteymix
Copy link
Contributor Author

arteymix commented Nov 8, 2016

I fixed the precedence of arguments to follow what generate_basic_compiler_args does.

I still can't figure why arguments are not passed on the initial meson.py call.

@arteymix arteymix force-pushed the wip/vala-fix-external-args branch 2 times, most recently from 93556ac to d8e7224 Compare November 12, 2016 02:29
@nirbheek
Copy link
Member

nirbheek commented Apr 9, 2017

What's the status of this now?

@arteymix
Copy link
Contributor Author

arteymix commented Apr 9, 2017

@nirbheek I need to rebase the work and see if -Dvala_args have already been addressed elsewhere

@arteymix arteymix force-pushed the wip/vala-fix-external-args branch from d8e7224 to b9080b8 Compare April 9, 2017 20:08
Fix precedence of arguments for Vala tasks and include those considered
external (i.e. '-Dvala_args' and 'VALAFLAGS').
@arteymix arteymix force-pushed the wip/vala-fix-external-args branch from b9080b8 to ad8a7ff Compare April 9, 2017 20:10
@arteymix
Copy link
Contributor Author

arteymix commented Apr 9, 2017

@nirbheek Everything is working! :)

I can't even get meson -Dc_args=-g to work. Is there any reason?

@nirbheek
Copy link
Member

nirbheek commented Apr 9, 2017

I didn't even know that was a thing, @jpakkane?

@arteymix
Copy link
Contributor Author

@jpakkane Is it possible to schedule this PR for 0.40?

@jpakkane
Copy link
Member

Please rebase so the tests are rerun. If they pass then we'll merge this.

@jpakkane jpakkane merged commit aa486f5 into mesonbuild:master Apr 22, 2017
@nirbheek
Copy link
Member

@arteymix could you look at #1645 next? Allowing VALAC to set the vala compiler.

@arteymix arteymix deleted the wip/vala-fix-external-args branch April 22, 2017 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants