https://pantsbuild.org/ logo
#general
Title
# general
n

narrow-vegetable-37489

02/16/2022, 12:21 PM
Hmm. Having some issues resolving the constraints file after adding a new dependency (
ring
) that I'm not really sure why it's happening from looking at the output. The constraints file comes from
poetry export
, and Poetry is able to install the dependency just fine. Logs in thread.
Untitled.txt
wirerope
is a dependency of
ring
, not sure why it's unable to be installed though.
wirerope==0.4.5
very much exists on pypi.
e

enough-analyst-54434

02/16/2022, 12:30 PM
The warning indicates wirerope was found but failed to build. Can you re-run with
--pex-verbosity=9
(I think 3 may actually be enough) and see if that reveals more?
n

narrow-vegetable-37489

02/16/2022, 12:35 PM
Ah, perfect. Thanks! Seems like wirerope has some custom code in their setup.py that crashes due to setuptools' version containing a hash in the end.
Copy code
ValueError: invalid literal for int() with base 10: '0+3acb925dd708430aeaf197ea53ac8a752f7c1863'
e

enough-analyst-54434

02/16/2022, 12:38 PM
This looks likely, although Pex vendors a newer setuptools than that: https://github.com/youknowone/wirerope/blob/81c533d6df479cae80f74b5c298c4236f98f0158/setup.py#L3
What version of Pants is this @narrow-vegetable-37489?
n

narrow-vegetable-37489

02/16/2022, 12:40 PM
Copy code
Running command python setup.py egg_info
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/_h/fr5ctz890wb5h87sgsvbdr5h0000gn/T/process-executionuTO3si/.tmp/pip-download-v1heqzg0/wirerope_8e7e469bacad47419c57647ad8aa5798/setup.py", line 3, in <module>
        assert tuple(map(int, setuptools.__version__.split('.')[:3])) >= (39, 2, 0), \
    ValueError: invalid literal for int() with base 10: '0+3acb925dd708430aeaf197ea53ac8a752f7c1863'
Forgot to include the trace as well.
2.10.0.dev2
, but tried it on latest RC as well.
e

enough-analyst-54434

02/16/2022, 12:40 PM
Aha, yeah.
n

narrow-vegetable-37489

02/16/2022, 12:43 PM
Found a relevant issue on their issue tracker: https://github.com/youknowone/wirerope/issues/8
Ok, yeah. I agree with that issue. The project should be switched to PEP-517 / PEP-518 to control setuptools bootstrap requirements.
That doesn't help you though! I'm not sure I have any great ideas here...
n

narrow-vegetable-37489

02/16/2022, 12:48 PM
No problem! It's not super critical, I can work around it until wirerope hopefully fixes the issue on their end. Thanks for the help though, very much appreciated! 🙂
e

enough-analyst-54434

02/16/2022, 5:36 PM
Ok, I sent a thorough fix. Hopefully they accept, but the project also seems dormant: https://github.com/youknowone/wirerope/pull/17
🙌 2
5 Views