rusackas commented on code in PR #44767: URL: https://github.com/apache/superset/pull/44767#discussion_r4126881691
########## requirements/development.in: ########## @@ -45,3 +45,14 @@ # DockerContainer plus whatever driver its own extra above already # provides. testcontainers[cockroachdb,cratedb,mongodb,mssql,mysql,oracle,postgres,trino]>=4.15.0,<5 + +# gevent (pulled in via the `gevent` extra above) has no direct version +# requirement on zope.interface, so the resolver was free to pick the +# oldest version satisfying gevent's own loose constraint: 5.4.0. That +# version's metaclass treats every class attribute as an interface member, +# including `__firstlineno__`, the implicit attribute Python 3.13 (PEP 626 +# byte-code line tracking) adds to every class -- so importing gevent +# (and anything that imports it, e.g. the gunicorn+gevent worker class) +# raises `zope.interface.exceptions.InvalidInterface` outright on 3.13. +# 7.2 is the first release with explicit handling for it. +zope.interface>=7.2 Review Comment: gevent doesn't pin it itself, so the resolver grabbed the oldest version satisfying gevent's own loose constraint (5.4.0), which chokes on 3.13's `__firstlineno__`. 7.2 is the first release with explicit handling for it, hence the floor there. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
