Skip to content

Conversation

@tintou
Copy link
Contributor

@tintou tintou commented Jun 27, 2025

When giving a dependency object as requires, allow to use the dependency from a subproject (that is an InternalDepdency).

@tintou tintou requested review from jpakkane and xclaesse as code owners June 27, 2025 20:48
@tintou tintou force-pushed the tintou/pkgconfig-subproject branch 4 times, most recently from c06e932 to e0249bd Compare June 27, 2025 21:28
@tintou

This comment was marked as resolved.

@tintou tintou force-pushed the tintou/pkgconfig-subproject branch 5 times, most recently from eb0a216 to 9383671 Compare June 30, 2025 19:38
…ency

When giving a dependency object as requires, allow to use the dependency from a
subproject (that is an InternalDepdency).
@tintou tintou force-pushed the tintou/pkgconfig-subproject branch from 9383671 to aff94c5 Compare June 30, 2025 19:49
@jpakkane jpakkane merged commit a82c545 into mesonbuild:master Aug 1, 2025
34 checks passed
@bonzini bonzini added this to the 1.9 milestone Aug 1, 2025
bgilbert added a commit to bgilbert/meson that referenced this pull request Sep 3, 2025
@bgilbert
Copy link
Contributor

bgilbert commented Sep 3, 2025

Docs update in #14994.

@eli-schwartz
Copy link
Member

We should go one step further and emit a FeatureNew based warning when this code path is taken, so that people can easily detect when they are relying on this.

bgilbert added a commit to bgilbert/meson that referenced this pull request Sep 3, 2025
@bgilbert
Copy link
Contributor

bgilbert commented Sep 3, 2025

Added to #14994.

bgilbert added a commit to bgilbert/meson that referenced this pull request Sep 4, 2025
bgilbert added a commit to bgilbert/meson that referenced this pull request Sep 10, 2025
jpakkane pushed a commit that referenced this pull request Sep 10, 2025
eli-schwartz pushed a commit that referenced this pull request Sep 22, 2025
Added in #14750 for 1.9.0.

Also add FeatureNew.

(cherry picked from commit 3734ff4)
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.

5 participants