Skip to content

avoid meson 1.5 for now#397

Merged
h-vetinari merged 2 commits into
conda-forge:mainfrom
h-vetinari:meson
Jul 11, 2024
Merged

avoid meson 1.5 for now#397
h-vetinari merged 2 commits into
conda-forge:mainfrom
h-vetinari:meson

Conversation

@h-vetinari

Copy link
Copy Markdown
Member

whew, 3.9 is really resisting green CI on main 😅

@conda-forge-webservices

Copy link
Copy Markdown

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@dopplershift

Copy link
Copy Markdown
Member

This seems to be failing on osx arm64, not sure why it's trying to run an exe though.

@h-vetinari

Copy link
Copy Markdown
Member Author

This is due to some as-yet undetermined change in meson 1.5.0, which was just released. I thought it was the misspelling of pkg-config, but it seems to be something else. If it's coming from the meson_cross_file.txt, then we'll have to fix it in the compiler activation, c.f. conda-forge/clang-compiler-activation-feedstock#112

@h-vetinari

Copy link
Copy Markdown
Member Author

I thought it was the misspelling of pkg-config

Man, whoever thought it was a good idea to have both pkgconfig and pkg-config.... Ugh. So the binary is named (nowadays at least) pkg-config, but the actual metadata for it still lives under $PREFIX/lib/pkgconfig.

@h-vetinari h-vetinari force-pushed the meson branch 4 times, most recently from ae684f7 to 4f6eb25 Compare July 11, 2024 02:40
@h-vetinari h-vetinari changed the title fix pkg-config path debug meson issues on osx-arm64 Jul 11, 2024
@h-vetinari h-vetinari changed the title debug meson issues on osx-arm64 avoid meson 1.5 for now Jul 11, 2024
@h-vetinari h-vetinari merged commit 414fc00 into conda-forge:main Jul 11, 2024
@h-vetinari h-vetinari deleted the meson branch July 11, 2024 04:23
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