Open woodruffw opened 1 year ago
Hi there; yeah, we have quite a lot of legacy pinning to get around issues we had with pip when they were transitioning their resolver during the pip 21+ days. pip 23 fundamentally broke something (that I can't remember) and so we pinned to move forward at that time.
I also notice that you are using Python 3.11, and I'm almost certain we haven't tested charm-tools with py3.11; I doubt you want to carry a patch, but it would be interesting to see if relaxing the pip constraint with py3.11 yields a working charm-tools. Hmm.
That's helpful context, thank you!
I think it should be possible for the formula to request a lower Python version, e.g. the 3.10 series. I'll look into that.
(If 3.11 is untested on your end, it might be good to enforce that with a python_requires
cap in your setup.py
🙂 -- that would have caused Homebrew's package to fail earlier.)
(If 3.11 is untested on your end, it might be good to enforce that with a
python_requires
cap in yoursetup.py
slightly_smiling_face -- that would have caused Homebrew's package to fail earlier.)
I mean I could do, but then you'd come back with a bug report saying "doesn't support Python 3.11" :)
We probably need to get it tested with py3.11 and unpin pip (or do a selective python_version pin for py3.10) as we ought to be marching forward and supporting the newer python versions. Also, add the python_requires
capability would be a good idea, so thanks for the suggestion!
I mean I could do, but then you'd come back with a bug report saying "doesn't support Python 3.11" :)
That's fair, we probably would! But think you could state it as a matter of policy, and we'd pin your package to python@3.10
on the Homebrew side 😉
(And thank you for the fast response here! I believe we can also temporarily fix this on the Homebrew side, until you're ready as an upstream to support 3.11 🙂)
Hi there, this is an upstream packaging report for
charm-tools
version3.0.6
.It looks like you're currently constraining
pip < 23
, which will make your package uninstallable on any host with an upgraded pip (specifically, one released this year).Example failure: https://github.com/Homebrew/homebrew-core/pull/131661
And the actual failure:
And a permalink to
charm-tools
' own constraint:https://github.com/juju/charm-tools/blob/1847cbeba3700c6175f3dd494f9a66e8fb541a82/setup.py#L55
Unless there's a specific reason why you're listing
pip
in yourinstall_requires
, my recommendation would be to omit it entirely: the Python runtime is meant to carry it. If for whatever reason it doesn't and you absolutely do need to list it, leaving it open ended on the upper bound is probably fine.