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.

Sign up for GitHub

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

Jump to bottom

Build error with PyYAML 5.4.1 #1507

Closed
jdeanwallace opened this issue Jul 18, 2023 · 2 comments · Fixed by #1508
Closed

Build error with PyYAML 5.4.1 #1507

jdeanwallace opened this issue Jul 18, 2023 · 2 comments · Fixed by #1508
Assignees
@jdeanwallace

Comments

@jdeanwallace
Copy link
Contributor

One of my PR builds started failing all of a sudden with the following error:

ERROR: Could not find a version that satisfies the requirement pyyaml==5.4.1
ERROR: No matching distribution found for pyyaml==5.4.1

I found this issue that suggests the error coincides with the release of Cython 3.

Someone suggested the following fix:

You can also upgrade to 6.0.1, which pins the Cython < 3.0.0.
@jdeanwallace jdeanwallace mentioned this issue Jul 18, 2023
Merged
@jdeanwallace jdeanwallace self-assigned this Jul 18, 2023
@mtlynch
Copy link
Contributor

mtlynch commented Jul 18, 2023

@jdeanwallace - I think your work on #1352 saved customers from encountering this bug in the field. Glad we got that in!

@jdeanwallace
Copy link
Contributor Author

I think your work on #1352 saved customers from encountering this bug in the field.

*our work

Oh yeah! 🤠

jdeanwallace added a commit that referenced this issue Jul 19, 2023
@jdeanwallace
Upgrade to PyYAML 6.0.1 (#1508)
ec45ba7 
Resolves #1507

The release of [Cython `3` broke PyYAML
`5.4`](yaml/pyyaml#724). This PR upgrades to
PyYAML `6.0.1` which pins `Cython < 3.0.0`.

Notes
1. While bootstrapping the molecule testing environment, it was
[complaining about the following dependency
conflict](https://app.circleci.com/pipelines/github/tiny-pilot/tinypilot/3695/workflows/1e8382a7-5a08-4a64-a87a-a5af41916c70/jobs/22051?invite=true#step-105-155):
    ```bash
    The conflict is caused by:
        The user requested PyYAML==6.0.1
        molecule 3.2.3 depends on PyYAML<6 and >=5.1
    ```
So based on [someone's suggestion from the original GitHub
issue](yaml/pyyaml#724 (comment)),
I pinned `pyyaml` to version `5.3.1`. However, I only did this for the
[molecule
dependencies](https://github.com/tiny-pilot/tinypilot/pull/1508/files#diff-0852a7343763d1af5bb207aac9c7ac61e43aea1d676c9ab25ba6ae7503804578).

<a data-ca-tag
href="https://codeapprove.com/pr/tiny-pilot/tinypilot/1508"><img
src="https://codeapprove.com/external/github-tag-allbg.png" alt="Review
on CodeApprove" /></a>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdeanwallace jdeanwallace

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Upgrade to PyYAML 6.0.1 tiny-pilot/tinypilot
2 participants
@jdeanwallace @mtlynch