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

Publish wheels #2

Closed
maxstrobel opened this issue Jul 12, 2023 · 11 comments · Fixed by #3 or #4
Closed

Publish wheels #2

maxstrobel opened this issue Jul 12, 2023 · 11 comments · Fixed by #3 or #4
Labels
completed question Further information is requested todo

Comments

@maxstrobel
Copy link

Hi @rathaROG,

thanks for you efforts to make lap easier distributable - I really appreciate it!

image

One question: Could it be that the wheels are not published to pypi.org? If I try to install if (on any platform) it downloads the .tar.gz & tries to build it locally (on the pypi.org webpage I find also only the .tar.gz).

Is it possible that the publish workflow (https://github.com/rathaROG/lapx/blob/main/.github/workflows/build_pypi.yaml#L25) does not do, what it should do?

@rathaROG rathaROG added the question Further information is requested label Jul 12, 2023
@rathaROG
Copy link
Owner

rathaROG commented Jul 12, 2023

PyPI has strict binary distribution format like PEP 427 and PEP 491, etc., so it's hard to publish Cython wheels. It does not accept some of my wheels like ...-linux_x86_64.whl, and that's why I only published the source distribution tar.gz as my setup.py works fine on all platforms. Honestly, I haven't yet checked further on how to deal with this wheel naming for distribution, and it would be nice if you can help me with this. As it is now, you can install the right wheel .whl directly from releases like for Python 3.9 on Windows:

pip install https://github.com/rathaROG/lapx/releases/download/v0.5.2.post1/lapx-0.5.2.post1-cp39-cp39-win_amd64.whl

Maybe I can publish all wheels for Windows and macOS as they already have the acceptable names... 🤔

@maxstrobel
Copy link
Author

Yes, I saw the releases & tested them under Windows, Debian 11.x, RedHat 7.x & RedHat 8.x - all worked out of the box 🎉 (just tested with the simple usage example, but I guess that's enough)

I agree Windows & macOS should be already ready as they are.
For Linux: Probably we can go with manylinux (https://github.com/pypa/manylinux) - manylinux2014 could be a good candidate.

What do you think?

@rathaROG
Copy link
Owner

rathaROG commented Jul 13, 2023

Yes, I saw the releases & tested them under Windows, Debian 11.x, RedHat 7.x & RedHat 8.x - all worked out of the box 🎉 (just tested with the simple usage example, but I guess that's enough)

I agree Windows & macOS should be already ready as they are. For Linux: Probably we can go with manylinux (https://github.com/pypa/manylinux) - manylinux2014 could be a good candidate.

What do you think?

It looks interesting. I will try this weekend.

@duane-edgington
Copy link

duane-edgington commented Jul 13, 2023

I am trying to install a rather complex set of packages to support a particular github repository. A requirement is to install lapx wheel using pip. I am on an MacOS 13.4.1 system on M1 Max (arm64), Python 3.11.4. It fails the build. I will attach the error log here. What should I do to install the package?
lapx-error.txt

@rathaROG
Copy link
Owner

I am trying to install a rather complex set of packages to support a particular github repository. A requirement is to install lapx wheel using pip. I am on an MacOS 13.4.1 system on M1 Max (arm64), Python 3.11.4. It fails the build. I will attach the error log here. What should I do to install the package? lapx-error.txt

Hi, I don't have MacBook arm64 M1 or M2 to test, but I can tell you that lapx needs C++ compiler to build Cython extension. According to your log, the path of your clang compiler is not set or not recognized.

xcrun: error: invalid active developer path (/Library/Developer/CommandLineTools), missing xcrun at: /Library/Developer/CommandLineTools/usr/bin/xcrun
      error: command '/usr/bin/clang' failed with exit code 1

Maybe you should check if you can compile any C++ file from your terminal. This might be helpful for you: https://apple.stackexchange.com/questions/414622/installing-a-c-c-library-with-homebrew-on-m1-macs

@duane-edgington
Copy link

Perfect! Thank you. I had just the day before upgraded the OS (major from 12.something to the latest 13.4.1) and I did not check to see if the Mac xcode command line tools (which includes the CLANG and C++ compilers) had come along or needed to be installed. When I install the xcode command line tools afresh, the build worked perfectly. Thank you so much for quick response!

@rathaROG
Copy link
Owner

Perfect! Thank you. I had just the day before upgraded the OS (major from 12.something to the latest 13.4.1) and I did not check to see if the Mac xcode command line tools (which includes the CLANG and C++ compilers) had come along or needed to be installed. When I install the xcode command line tools afresh, the build worked perfectly. Thank you so much for quick response!

I'm glad to hear that! Thanks for letting me know.

@rathaROG rathaROG linked a pull request Jul 18, 2023 that will close this issue
@rathaROG
Copy link
Owner

rathaROG commented Jul 18, 2023

Not yet finished.. I will try cibuildwheel under a new branch https://github.com/rathaROG/lapx/tree/cibuildwheel

@rathaROG rathaROG reopened this Jul 18, 2023
@rathaROG rathaROG added the todo label Jul 18, 2023
@rathaROG rathaROG linked a pull request Jul 18, 2023 that will close this issue
@rathaROG
Copy link
Owner

rathaROG commented Jul 19, 2023

@maxstrobel

It's now completed!

Try the new -> v0.5.3
Check PyPI -> https://pypi.org/project/lapx/#files

Tested on latest Linux Mint 21.2

image

Tested on WSL Ubuntu

image

Tested on native Windows 11

image

@maxstrobel
Copy link
Author

Looks great - thanks for the efforts.

I did already some basic tests & everything seems to work as expected :)

@rathaROG
Copy link
Owner

Looks great - thanks for the efforts.

I did already some basic tests & everything seems to work as expected :)

Thank you very much for the tests!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
completed question Further information is requested todo
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants