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

2.1.0 broken on Python 3.4 #29

Closed
avylove opened this issue Oct 14, 2021 · 6 comments
Closed

2.1.0 broken on Python 3.4 #29

avylove opened this issue Oct 14, 2021 · 6 comments
Assignees

Comments

@avylove
Copy link

avylove commented Oct 14, 2021

2.1.0 seems to be broken on Python 3.4

  File "/home/runner/work/blessed/blessed/.tox/py34/lib/python3.4/site-packages/apipkg/__init__.py", line 69, in initpkg
    mod = _initpkg_py3(mod, pkgname, exportdefs, attr=attr)
  File "/home/runner/work/blessed/blessed/.tox/py34/lib/python3.4/site-packages/apipkg/__init__.py", line 141, in _initpkg_py3
    mod.__class__ = ApiModule
TypeError: __class__ assignment: only for heap types

For my purposes I can probably pin to 2.0.1. However, the readme for this package specifically calls out support fr 3.4+, but I notice 3.4 is not being tested. 3.4 can be tested in GH actions by pinning os: ubuntu-18.04. Please fix or drop support for 3.4.

@webknjaz
Copy link
Member

@webknjaz
Copy link
Member

@webknjaz
Copy link
Member

@RonnyPfannschmidt if it's not desired to keep the support for Python 3.4, python_requires needs to be updated and the previous release yanked. If it's an unintended bug — maybe it could be fixed?

cc @GlenWalker

webknjaz added a commit to cherrypy/cheroot that referenced this issue Nov 25, 2021
@RonnyPfannschmidt
Copy link
Member

Unintended, im not sure when I can get to fixing

@webknjaz
Copy link
Member

@RonnyPfannschmidt maybe you could still update the metadata, republish a new patch release for >=3.5, and yank the broken one? As a damage control measure. Then #31 could re-enable 3.4 and lower python_requires back to what it was.

@RonnyPfannschmidt
Copy link
Member

yanked after a 2.1.1 with patch

the next release will drop support for all dead python versions

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

No branches or pull requests

3 participants