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

Could you add libpython.a files to the AppImages? #69

Open
mattaw opened this issue Oct 8, 2023 · 2 comments
Open

Could you add libpython.a files to the AppImages? #69

mattaw opened this issue Oct 8, 2023 · 2 comments

Comments

@mattaw
Copy link

mattaw commented Oct 8, 2023

I was just attempting to build an older uwsgi against a manylinux python 3.10 on RHEL7 (I am not allowed to upgrade it, sadly).

One of the issues I ran into was a missing libpython.a library. This was encountered in the original manylinux docker images, and to address it the manylinux image now includes them in compressed form: pypa/manylinux#1250 as it was less than 1% image increase I believe.

Could you consider including the appropriate compressed libpython.a in the image? If so I would be happy to create the PR to add it to the build and documentation to tell people where it is and how to get at it. Considering AppImages are by their nature compressed it may be possible to simply include the libpython.a file uncompressed in the right place anyway.

If you could point me to the file that bundles the files into squashfs I could simply include a decompression of the existing libpython.a files and copy the right one to the right place to automatically include it.

Thank you for all your hard work getting python going on ancient systems!

@niess
Copy link
Owner

niess commented Oct 10, 2023

Hello @mattaw ,

thank you for pointing this out. I was not aware of this issue. This looks reasonable to me if indeed the AppImage size does not significantly increase. But, this might not be true in the case of AppImages. I mean 1% of a Docker image (GBs?) is likely much more than 1% of a Python AppImage (~20 MB).

In any case, if you like to experiment with this, you could comment this line, which actually removes the static library from the AppImage.

My fear is that the AppImage size might double doing so :(

@mattaw
Copy link
Author

mattaw commented Oct 12, 2023

Thanks for identifying that line, I tried to find it but for some reason could not get the right search terms!
Do you think it worth choosing a suitable manylinux-appimage, setup a vanilla packaging run of it using the instructions here, reproduce one of your releases (mostly) using these instructions:

https://python-appimage.readthedocs.io/en/latest/apps/#manylinux-python-appimages

and then edit that line to leave libpython.a in place and report back the size differences?

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

2 participants