hey all I am having trouble generating a lock-file...
# generalw
wooden-ability-49079
10/17/2023, 4:36 PMhey all I am having trouble generating a lock-file that due a dependency that does not seem to play nice with the
--style=universalstyle=stricte
enough-analyst-54434
10/17/2023, 6:07 PM@wooden-ability-49079, I bet you can't even install that package with Pip. What Python version are you targeting? Unless you use a VCS requirement, that project should only work with Python 3.5 since it publishes no source distribution and just a py35 wheel for each of the two versions.
enough-analyst-54434
10/17/2023, 6:10 PMIn other words, locking vs Pip, etc is not the issue. The issue is Python 3.5 only. So do you have interpreter constraints configured as
==3.5.*enough-analyst-54434
10/17/2023, 6:10 PMThat would be required here.
w
wooden-ability-49079
10/17/2023, 11:59 PMah right, didn't notice the python constriant, surprsingly pip just works, we are actually targetting 3.9+
Copy code
❯ python --version
Python 3.11.6
❯ pip install python-forge==18.6.0
Looking in indexes: <https://pypi.org/simple/>
Collecting python-forge==18.6.0
Using cached python_forge-18.6.0-py35-none-any.whl (31 kB)
Installing collected packages: python-forge
Successfully installed python-forge-18.6.0
❯ pip --version
pip 23.2.1 from /Users/user/throwaway/lib/python3.11/site-packages/pip (python 3.11)wooden-ability-49079
10/18/2023, 12:00 AMis there a way to override that constraint and have pants installed the dependency regardless, or perhaps I can create an override for that particular dependency? how would one get around this?
e
enough-analyst-54434
10/18/2023, 3:40 AM@wooden-ability-49079, I still don't grok what's going on here exactly, but can you please reveal the interpreter constraints you configured? Also potentially relevant is the Pants version you are using.
enough-analyst-54434
10/18/2023, 4:01 AMOk, although this handles the
abi3noneenough-analyst-54434
10/18/2023, 4:05 AMIOW
py35-none-anyenough-analyst-54434
10/18/2023, 4:05 AM@wooden-ability-49079 I'm out of country and AFK through November 2nd but can file and fix then.
w
wooden-ability-49079
10/18/2023, 4:06 AMgotcha, happy to collaborate on this once I understand what is actually broken, apologize I still don't fully understand what's going on
e
enough-analyst-54434
10/18/2023, 4:07 AMFor clarity it would be awesome if you did provide ICs and Pants version.
w
wooden-ability-49079
10/18/2023, 4:07 AM Copy code
❯ pants --version
21:07:31.54 [INFO] Initialization options changed: reinitializing scheduler...
21:07:35.25 [INFO] Scheduler initialized.
2.17.0wooden-ability-49079
10/18/2023, 4:08 AM Copy code
[python]
interpreter_constraints = ['==3.9.*']
enable_resolves = truewooden-ability-49079
10/18/2023, 4:08 AMbut this fails on 3.11 as well as constraint, let me try 3.5
e
enough-analyst-54434
10/18/2023, 4:10 AMOk, thanks. I'm pretty sure the issue is the Pex logic I identified above. Roughly zero projects publish a wheel with that style tag plus no sdist; so it makes sense this wouldn't show up for 2 years.
enough-analyst-54434
10/18/2023, 4:12 AMIt will work for a
==3.5.*w
wooden-ability-49079
10/18/2023, 4:13 AMagreed
e
enough-analyst-54434
10/18/2023, 4:13 AMIt will work, but other things fail most likely. Many projects floor at 3.7 now.
w
wooden-ability-49079
10/18/2023, 4:15 AMexactly, I can't realistically set the constraint to 3.5, but I was hoping to understand what is going here, as this particular issue is kind of blocking me, would it be helpful to create a gh issue?
e
enough-analyst-54434
10/18/2023, 4:15 AMYes, thank you.
enough-analyst-54434
10/18/2023, 4:15 AMYou can unblock with a VCS requirement.
enough-analyst-54434
10/18/2023, 4:16 AMIs this a direct dep or transitive?
w
wooden-ability-49079
10/18/2023, 4:16 AMtransitive unfortuantely
wooden-ability-49079
10/18/2023, 4:16 AMVCS as in point to the github repo of the dep?
e
enough-analyst-54434
10/18/2023, 4:17 AMWell, you can add a top level req anyhow with a comment explaining the workaround.
enough-analyst-54434
10/18/2023, 4:17 AMYes, just a sec...
enough-analyst-54434
10/18/2023, 4:19 AMpython_forge @ git+<https://github.com/dfee/forge@SHA|https://github.com/dfee/forge@SHA>enough-analyst-54434
10/18/2023, 4:19 AMReplace
SHAenough-analyst-54434
10/18/2023, 4:22 AMOr, if you trust they don't mutate release tags, use
v18.6.0w
wooden-ability-49079
10/18/2023, 4:24 AMgotcha, yes that worked!! nice thanks a heap for your help @enough-analyst-54434, I'll open gh ticket, would love to publish a patch for this, but I super new at pants, so I am still learning how this all works
e
enough-analyst-54434
10/18/2023, 4:25 AMNo worries and thank you. This is not a Pants issue, it's a Pex issue. Please file here: https://github.com/pantsbuild/pex
w
wooden-ability-49079
10/18/2023, 4:25 AM👍
wooden-ability-49079
10/18/2023, 4:34 AMe
enough-analyst-54434
10/19/2023, 4:56 AMThanks @wooden-ability-49079