-
-
Notifications
You must be signed in to change notification settings - Fork 291
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pex.resolver.Untranslateable
exception in pex>=1.4.0
#492
Comments
it looks like what's happening here is that pex is attempting to translate pex usually falls back to this mode only if no linux-compatible bdists are found in the resolver path. it does not look like does your custom index provide a linux-compatible bdist for wrapt-1.10.11 to support this build? if so, does this improve if you pass an extended platform of |
(and if so, it seems we could improve the behavior here by widening the ABI permutations generated for non-specific platform specifications. IIRC, right now that keys off of the currently executing interpreters ABI, which would typically be different from OSX ( |
pants does not yet consume pex 1.4.x, so there's no way these two particular issues could be related. but in general, https://github.com/pantsbuild/pants is the best place to file and work on pants related tickets. |
Sorry about that; comment deleted. Well, that explains why pants hates my manylinux wheel. Thanks for the heads-up |
@DrumMachineLearning no worries. 1.4.x was just released and contains a pretty invasive change to the way bdist resolution works, so we're letting that bake/stabilize upstream (here) first - then should be pulling that into pants shortly (sometime in the 1-2 weeks). |
@pavdmyt can you confirm that the above suggestion (passing |
@kwlzn sorry for delay, here is more info. I was inspecting this issue at OSX, but initially it started failing at our "build server" (container based on alpine linux) where At my custom index I have pre-built wheels for OSX and Ubuntu 16.04 (the intention is to run resulted pex application at both systems):
I've replaced Thanks! |
@pavdmyt no worries! that's great - thanks for confirming. I have a fix in mind to improve the ergonomics here so that it works as it did in 1.3.x for improved backward compatibility - will try to circle back to that within a week to cut 1.4.3. until then, would use the extended platform form above as a workaround (or permanent change as that form will continue to work). |
pex 1.4.3 is now cut, so the behavior here should be back to what you'd see with 1.3.x. let me know if you hit any further issues and I'd be happy to take a look. |
FYI we had a similar issue and bumping the version from 1.4.1 to 1.4.3 fixed the problem for us. Thanks! |
I'm building PEX for my project with the following
sh
script:Contents of
requirements-pex.txt
:Using
pex 1.4.0
orpex 1.4.1
having the following issue (everything works fine withpex 1.3.2
):The text was updated successfully, but these errors were encountered: