pantsbuild #general

Channels

welcome

github-notifications

announce

faint-grass-11531

03/28/2017, 10:32 PM

oooh gotcha

rough-minister-58256

03/28/2017, 10:32 PM

the find links repos need to have trailing /‘s too

rough-minister-58256

03/28/2017, 10:32 PM

possible that it’s an internal vs external divide thing - not sure of other cases where people are using a mixed mode like that

rough-minister-58256

03/28/2017, 10:33 PM

we pre-build all of our deps as wheels and then do the equiv of

pex -f <http://our-wheelhouse/wheels/>

for all builds

rough-minister-58256

03/28/2017, 10:33 PM

may be a good pattern for you guys to adopt - gives you more control and consistency over the products that actually get resolved

faint-grass-11531

03/28/2017, 10:34 PM

yeah things are kind of a mess right now 😐

rough-minister-58256

03/28/2017, 10:34 PM

you can prebuild transitive wheels easily with pip:

pip wheel --no-cache-dir —wheel-dir=./wheelhouse dep==1.0.0 dep2==2.0.0

faint-grass-11531

03/28/2017, 10:35 PM

and then would i do

-f ./wheelhouse

rough-minister-58256

03/28/2017, 10:35 PM

yeah or just copy those outputs to your pypirepo url or wherever

faint-grass-11531

03/28/2017, 10:35 PM

hmm, okay

faint-grass-11531

03/28/2017, 10:35 PM

thank you 🙂

rough-minister-58256

03/28/2017, 10:35 PM

np!

rough-minister-58256

03/28/2017, 10:36 PM

but note that it is possible there’s also a pex bug lurking here too 🙂

faint-grass-11531

03/28/2017, 10:36 PM

hmm how do i investigate that further?

rough-minister-58256

03/28/2017, 10:37 PM

you’re on the right track trying to get to a minimal repro of the issue

faint-grass-11531

03/28/2017, 10:38 PM

okay 🙂 i'll keep you updated

rough-minister-58256

03/28/2017, 10:38 PM

👍

faint-grass-11531

03/28/2017, 10:42 PM

haha guess what! we just dont have oauthlib 0.6.3 in our pypi mirror >.<

faint-grass-11531

03/28/2017, 10:42 PM

wait no we do have it

faint-grass-11531

03/28/2017, 10:42 PM

i'm just blind

faint-grass-11531

03/28/2017, 10:42 PM

... investigation continues

rough-minister-58256

03/28/2017, 10:44 PM

one other thing you could try here to determine whether it’s a pex bug or environment issue - based on past resolver issues - would be to transitively pin that req in your top level target deps

rough-minister-58256

03/28/2017, 10:45 PM

so declare

oauthlib==0.6.3

in your requirements.txt and add

3rdparty/python:oauthlib

into your top level targets deps

faint-grass-11531

03/28/2017, 10:45 PM

ah ok. oauthlib==0.6.3 is already in requirements.txt -- i'll try the rest

rough-minister-58256

03/28/2017, 10:45 PM

so it’ll make it in the first round vs transitive rounds of resolves

faint-grass-11531

03/28/2017, 11:12 PM

ooo that worked ! it actually ran a test 😄 (which failed, but that's fixable!)

faint-grass-11531

03/28/2017, 11:13 PM

so the only difference was that i added the

3rdparty/python:oauthlib

dep in the BUILD file of the project that required

auth-client

, and afterwards the dependencies were able to resolve

faint-grass-11531

03/28/2017, 11:14 PM

maybe a real PEX bug then? I'm not sure how to report it given how it depends on our weird internal repos

faint-grass-11531

03/28/2017, 11:16 PM

ALSO i checked in with my team and they love wheels, we just don't have time to set it up right now. Wheels don't have any way of specifying what linux distro it's for so either we'd have to have separate wheels repos for different distros, or put the name of the distros in the names of the wheel packages or something

faint-grass-11531

03/28/2017, 11:40 PM

@rough-minister-58256 YOU ARE A HERO!!

Copy code

===== 127 passed, 7 skipped in 2.01 seconds ======

23:39:16 11:00   [complete]
               SUCCESS

🎉 🎉 🎉