Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Relax pyyaml versioning during building #125

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

Merged
merged 1 commit into from
Aug 20, 2022

Conversation

masklinn
Copy link
Contributor

e27d50d moved the yaml dependency from runtime to dev and updated the version from 5.1 to 5.4 (not 6.0 as that dropped Python 2.7 compatibility), but that turns out to cause issues for people who can't / won't download pyyaml on the fly and require building to work with preinstalled packages, as building regexes now refuses to work with a locally available 6.0.

Remove version bound on pyyaml (though a >= bound might also have worked), but add a bunch of tests to ensure the regexes build on all supported versions of Python, and pyyaml (also reinstate 5.1 for that purpose only).

Fixes #121

@masklinn masklinn force-pushed the fix-yaml-build-dependency branch 3 times, most recently from cfa0686 to fa38d4a Compare August 20, 2022 14:11
e27d50d moved the yaml dependency
from runtime to dev and updated the version from 5.1 to 5.4 (not 6.0
as that dropped Python 2.7 compatibility), but that turns out to cause
issues for people who can't / won't download pyyaml on the fly and
require building to work with preinstalled packages, as building
regexes now refuses to work with a locally available 6.0.

Remove version bound on pyyaml (though a `>=` bound might also have
worked), but add a bunch of tests to ensure the regexes build on all
supported versions of Python, and pyyaml (also reinstate 5.1 for that
purpose only).

Fixes ua-parser#121
@masklinn masklinn force-pushed the fix-yaml-build-dependency branch from fa38d4a to 911624c Compare August 20, 2022 14:13
@masklinn masklinn merged commit 4e26047 into ua-parser:master Aug 20, 2022
@masklinn masklinn deleted the fix-yaml-build-dependency branch August 20, 2022 14:16
masklinn added a commit to masklinn/uap-python that referenced this pull request Aug 20, 2022
Followup to ua-parser#125 as it seems unnecessary to keep 3.10 on an outdated
non-binary-wheel version. Especially since it's just for parsing test
data.

Allow the test suite to use whatever the latest supported pyyaml is
for it. Also install libyaml for pypy.
masklinn added a commit to masklinn/uap-python that referenced this pull request Aug 20, 2022
Followup to ua-parser#125 as it seems unnecessary to keep 3.10 on an outdated
non-binary-wheel version. Especially since it's just for parsing test
data.

Allow the test suite to use whatever the latest supported pyyaml is
for it. Also install libyaml for pypy.
masklinn added a commit to masklinn/uap-python that referenced this pull request Aug 20, 2022
Followup to ua-parser#125 as it seems unnecessary to keep 3.10 on an outdated
non-binary-wheel version. Especially since it's just for parsing test
data.

Allow the test suite to use whatever the latest supported pyyaml is
for it. Also install libyaml for pypy.
masklinn added a commit to masklinn/uap-python that referenced this pull request Aug 20, 2022
Followup to ua-parser#125 as it seems unnecessary to keep 3.10 on an outdated
non-binary-wheel version. Especially since it's just for parsing test
data.

Allow the test suite to use whatever the latest supported pyyaml is
for it. Also install libyaml for pypy.
masklinn added a commit that referenced this pull request Aug 20, 2022
Followup to #125 as it seems unnecessary to keep 3.10 on an outdated
non-binary-wheel version. Especially since it's just for parsing test
data.

Allow the test suite to use whatever the latest supported pyyaml is
for it. Also install libyaml for pypy.
masklinn added a commit to masklinn/uap-python that referenced this pull request Aug 20, 2022
- removed version bound from `setup_requires`, allows building from
  source when only PyYaml 6.0 is available (and downloading 5.4 is
  not), ua-parser#121, ua-parser#125
- added type assertion to parser lookup, to make the error clearer on
  type error, ua-parser#122, ua-parser#123
- fixed `setup.py build_regexes` and `setup.py build` for non-in-place
  builds, ua-parser#127
@masklinn masklinn mentioned this pull request Aug 20, 2022
masklinn added a commit that referenced this pull request Aug 20, 2022
- removed version bound from `setup_requires`, allows building from
  source when only PyYaml 6.0 is available (and downloading 5.4 is
  not), #121, #125
- added type assertion to parser lookup, to make the error clearer on
  type error, #122, #123
- fixed `setup.py build_regexes` and `setup.py build` for non-in-place
  builds, #127
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.

setup.py install broke - because of yaml 6.0
1 participant
Morty Proxy This is a proxified and sanitized view of the page, visit original site.