Skip to content
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

Python Wheels #21

Open
armudgal opened this issue May 9, 2018 · 10 comments
Open

Python Wheels #21

armudgal opened this issue May 9, 2018 · 10 comments

Comments

@armudgal
Copy link

armudgal commented May 9, 2018

Hi,
I am working on wheels for this project and was writing a travis script for the same. I am currently targeting linux with PyPA's manylinux and is using their docker image.
I am facing some issues in the travis build and would like to hear your advice.
My fork: https://github.com/kira0204/v8py/tree/travis-wheels
Also, I am working with @buffer on this.

@tbodt
Copy link
Owner

tbodt commented May 9, 2018

  v8/out/native/obj.target/src/libv8_libplatform.a: could not read symbols: Malformed archive

I think this is because the version of v8 that was built outside Docker and cached by Travis is not compatible with the manylinux image. Try clearing the cache?

@tbodt
Copy link
Owner

tbodt commented May 9, 2018

I think the button to do that is under the "More Options" menu.

@armudgal
Copy link
Author

armudgal commented May 9, 2018

Sure thanks, will do so

@armudgal
Copy link
Author

armudgal commented May 10, 2018

Also, the built of v8 is done with the command python2.
The problem I am facing right now is that the CentOS5 docker image has Python2.4 as python2, and depot_tools need python2.7or higher to fetch --force v8. I am not sure what would be the best workaround here since we have different platforms over here.

One solution:
Instead of running the fetch --force v8 command, what we can do is run the command python2.7 depot_tools/fetch.py --force v8. Adding python2.7 to the PATH in the docker image would be a simple step then.

@armudgal
Copy link
Author

armudgal commented May 10, 2018

Update:
I had to include python2.7 in the path so that python2 resolved to python2.7.4 in this case.
The other issue being raised is here, the tar version in CentOS 5 is 1.15.1 and updating it is a very very weird option. Never thought building v8 only would be such a PITA in CentOS 5.

@tbodt
Copy link
Owner

tbodt commented May 10, 2018

Does CentOS 5 have a newer version of tar in the package repos? If not, you may end up having to compile it from source...

@armudgal
Copy link
Author

Hi, sorry for the late reply, no CentOS5 doesn't have a new version of TAR and compiling it from source is not the option because CentOS5 doesn't support it.
I put a pause on this because PyPA is updating the manylinux repo to run on a newer version on CentOS because of CentOS 5 reaching EOL a year back (pypa/manylinux#179).

The best option right now would be to wait for them to do it (which might take 3-4 weeks more) and then work on the new manylinux2010.

@desertkun
Copy link
Contributor

@armudgal just wondering who the manylinux thing go

@armudgal
Copy link
Author

@desertkun The manylinux2010 has been released, more info can be found here.

I didn't get much time to follow up on that though

@scottp-dpaw
Copy link

See #39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants