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

meson: Set same variables in declare_dependency() as in pkgconfig #231

Merged
merged 1 commit into from
Jan 20, 2021

Conversation

xclaesse
Copy link
Contributor

GTK currently fails to detect if epoxy has been built with EGL on
Windows when epoxy is a subproject. To fix that it needs to get that
information from the dependency variables.

@xclaesse
Copy link
Contributor Author

gnomesysadmins pushed a commit to GNOME/gtk that referenced this pull request Oct 15, 2020
epoxy_dep cannot be used in a configure time check when it comes from a
subproject. Use variables set in pc file instead.

This requires anholt/libepoxy#231.
gnomesysadmins pushed a commit to GNOME/gtk that referenced this pull request Oct 15, 2020
epoxy_dep cannot be used in a configure time check when it comes from a
subproject. Use variables set in pc file instead.

This requires anholt/libepoxy#231.
@xclaesse
Copy link
Contributor Author

@ebassi need to update Meson version in CI, do you know how?

@ebassi
Copy link
Collaborator

ebassi commented Oct 15, 2020

@xclaesse I'll have to update the Docker image we use on Travis.

@xclaesse
Copy link
Contributor Author

FYI, I fixed the inconsistency in meson: mesonbuild/meson#7853.

@ebassi
Copy link
Collaborator

ebassi commented Jan 20, 2021

We don't use Travis any more, so no more Docker images. Care to rebase on master?

GTK currently fails to detect if epoxy has been built with EGL on
Windows when epoxy is a subproject. To fix that it needs to get that
information from the dependency variables.

This requires Meson >=0.54.0 for setting variables in
declare_dependency().
@xclaesse
Copy link
Contributor Author

I miss gitlab's rebase button. Done manually :)

@ebassi ebassi merged commit 7ec320d into anholt:master Jan 20, 2021
@ebassi
Copy link
Collaborator

ebassi commented Jan 20, 2021

Thanks!

@xclaesse xclaesse deleted the meson-vars branch January 20, 2021 13:40
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.

2 participants