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

Fix use of VALAC so that it is respected everywhere #314

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rrthomas
Copy link
Contributor

@rrthomas rrthomas commented Oct 23, 2020

This PR comes in two parts:

  1. Fix Makefile, generator and valadoc-example-tester to use VALAC everywhere.
  2. Allow VALAC to be set in the environment. This makes it easy to use a different valac to test. Document in README.md that it must use a different API version from other installed versions.

@rrthomas
Copy link
Contributor Author

@ricotz said about this "note the vala version retrieval and library usage". I had a look at valadoc-example-tester.vala, and I can't see what this refers to, sorry!

@rrthomas rrthomas force-pushed the allow-different-valac branch 2 times, most recently from fffc988 to 4ac5645 Compare October 23, 2020 11:31
@rrthomas rrthomas changed the title WIP: Allow the use of a valac with a different name Fix use of VALAC so that it is respected everywhere Oct 23, 2020
@rrthomas rrthomas force-pushed the allow-different-valac branch 2 times, most recently from 826c6c3 to 87a24cf Compare October 23, 2020 13:41
@rrthomas
Copy link
Contributor Author

@ricotz said about this "note the vala version retrieval and library usage". I had a look at valadoc-example-tester.vala, and I can't see what this refers to, sorry!

@ricotz, ping? If I recall correctly, you weren't happy with the second commit, but the first seemed uncontroversial? If that's so, I can remove the second commit from this PR.

@rrthomas rrthomas force-pushed the allow-different-valac branch 2 times, most recently from fe42698 to c5b7b39 Compare January 15, 2021 20:38
This fixes some places where the VALAC setting was not respected
Document the need to use --disable-unversioned
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

Successfully merging this pull request may close these issues.

1 participant