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

ci: pin cargo-c version in pkg-config workflow #435

Merged
merged 1 commit into from
Jun 19, 2024
Merged

Conversation

cpu
Copy link
Member

@cpu cpu commented Jun 19, 2024

The recent 0.10.0 release seems to be causing a regression. Until we've had a chance to dig into the root cause let's pin the version in-use in CI to keep the build passing.

I also snuck in an update to print the pkg-config version in-use. I was interested in that data when I wasn't sure if it was the cargo-c update that changed things or if there was an unrelated system image update that bork'd it.

The recent release seems to be causing a regression. Until we've had
a chance to dig into the root cause let's pin the version in-use in CI
to keep the build passing.

I also snuck in an update to print the pkg-config version in-use. I was
interested in that data when I wasn't sure if it was the cargo-c update
that changed things or if there was an unrelated system image update
that bork'd it.
@cpu cpu self-assigned this Jun 19, 2024
@cpu cpu requested a review from ctz June 19, 2024 17:08
@cpu cpu merged commit 51f7cbf into rustls:main Jun 19, 2024
25 checks passed
@cpu cpu deleted the cpu-pin-cargo-c branch June 19, 2024 17:20
@cpu
Copy link
Member Author

cpu commented Jun 19, 2024

I think I figured out the issue. I'll revert this pinning + land a fix once I've gotten a better sense of whether the upstream change was intentional or not (and perhaps if there's a way we could make this test rig less brittle).

@lu-zero
Copy link
Contributor

lu-zero commented Jun 19, 2024

Sorry for the noise, try to set CARGOFLAGS=--libdir=lib it should return to the original behaviour

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.

3 participants