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

Release 8.11 breaks compatibility with Python 3.5 #578

Closed
mattbonnell opened this issue Nov 10, 2021 · 12 comments · Fixed by #583
Closed

Release 8.11 breaks compatibility with Python 3.5 #578

mattbonnell opened this issue Nov 10, 2021 · 12 comments · Fixed by #583

Comments

@mattbonnell
Copy link
Contributor

mattbonnell commented Nov 10, 2021

The new version makes use of f-strings, which were introduced in Python 3.6, making it incompatible with Python 3.5

File "<redacted>/.venv/lib/python3.5/site-packages/more_itertools/more.py", line 646

    f'Too few items in iterable (got {item_count})',

                                                  ^

SyntaxError: invalid syntax
@vfazio
Copy link

vfazio commented Nov 10, 2021

Note that 3.5 was dropped as of 8.6
https://github.com/more-itertools/more-itertools/blob/master/docs/versions.rst#860

as part of these PRs
#467
#469

@mattbonnell
Copy link
Contributor Author

mattbonnell commented Nov 10, 2021

The python-requires param in the setup.py needs to be bumped to prevent 3.5 users from automatically pulling the new version which they are no longer compatible with.

Docs

@andy-maier
Copy link

andy-maier commented Nov 10, 2021

@mattbonnell Right. The python-requires param is what pip evaluates. The classifiers are not evaluated by pip.

Circumvention for us was to exclude more-itertools 8.11.0 on Python 3.5 (see zhmcclient/python-zhmcclient#857).

10sr added a commit to 10sr/flake8-no-implicit-concat that referenced this issue Nov 11, 2021
mattbonnell added a commit to mattbonnell/more-itertools that referenced this issue Nov 11, 2021
@tysonite
Copy link

Can you remove release 8.11 on pypi? Or replace it with fixed version once #579 merged?

@webknjaz
Copy link

Oh, I was about to report the same issue. The runtime problem has been introduced in ba5240a#diff-d7dff201bd69500a573b7e96c52f8e625242d769a04621224112faf6e7bc63fdR600.

Instead of removing, I think it's okay if the maintainers use Yank, I think. These are the versions that need to be yanked: v8.11.2, v8.11.1, v8.11.0.

@webknjaz
Copy link

Cross-posting other incarnations of the same bug that has happened in the past: #272 + #355.

webknjaz added a commit to cherrypy/cheroot that referenced this issue Nov 23, 2021
`more-itertools` has a bug in their package metadata. They
have dropped the testing of Python 3.5 from their CI but
forgot to state this in the distribution package metadata.
This is why we need these conditionals to work around the
problem.

Refs:
* more-itertools/more-itertools#578
* more-itertools/more-itertools#579
@bbayles
Copy link
Collaborator

bbayles commented Nov 24, 2021

Bad news: I can't remove the old PyPI release.

Better news, I hope: I'll drop the f-strings and ship a new version.

@bbayles
Copy link
Collaborator

bbayles commented Nov 24, 2021

8.12.0 is now on PyPI with the f-strings dropped and 3.5 back in setup.py. Hopefully this fixes everybody's CI systems.

Somehow my GitHub notifications for non-mentions got turned off for this repository; apologies for not seeing this until today.

@hwalinga
Copy link
Contributor

Bad news: I can't remove the old PyPI release.

You should not remove it, but yank it instead:

https://www.python.org/dev/peps/pep-0592/

@bbayles
Copy link
Collaborator

bbayles commented Nov 24, 2021

I don't have the permissions on PyPI, I'm afraid.

@bbayles
Copy link
Collaborator

bbayles commented Nov 24, 2021

@erikrose, if you're available, could you yank 8.11.0 on PyPI?

.1 and .2 were GitHub-only releases for an unrelated RTD.

@erikrose
Copy link
Collaborator

erikrose commented Nov 25, 2021 via email

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 a pull request may close this issue.

8 participants