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

Attempt to fix CI failures related to PyYAML. #897

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

licquia
Copy link
Collaborator

@licquia licquia commented Apr 5, 2024

Due to a build incompatibility between PyYAML and Cython 3.0.0, builds of PyYAML from source to satisfy a dependency will fail without a workaround introduced in PyYAML 6.0.1. Therefore, we should insist on installing at least that version.

Fixes #867.

Due to a build incompatibility between PyYAML and Cython 3.0.0, builds
of PyYAML from source to satisfy a dependency will fail without a
workaround introduced in PyYAML 6.0.1.  Therefore, we should insist on
installing at least that version.

Fixes #867.

Signed-off-by: Jeff Licquia <[email protected]>
@licquia licquia requested a review from goneall April 5, 2024 02:28
@licquia
Copy link
Collaborator Author

licquia commented Apr 5, 2024

Since this has passed the exact same check that failed before, I'm going to merge. If it shows up in other branches, I expect the same fix will take care of the problem there too.

@licquia licquia merged commit 9039f27 into development/v3.0 Apr 5, 2024
2 checks passed
@licquia licquia deleted the pyyaml-build-workaround branch April 5, 2024 02:31
@goneall
Copy link
Member

goneall commented Apr 5, 2024

Thanks @licquia!

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