[Needs review] Fix python site-packages dir#3916
[Needs review] Fix python site-packages dir#3916NTULINUX wants to merge 1 commit intoLinuxCNC:masterfrom
Conversation
|
Looks like this needs to be fixed to account for Debian..
Ah, looks like various parts of the tree are also hard-coded to use Did not see these. Any tips? The issue is that depending on the exact version of python that debian uses will affect the installation path, which will vary from each debian release, that's the tricky part to make this work across all versions. Not sure on a clean way to avoid the hard-coded variables either, as those will also change. |
|
Is there a distro independent way to find out where python packages need to be installed? |
|
Well that's what this should do: https://github.com/LinuxCNC/linuxcnc/blob/master/src/m4/ax_python_devel.m4#L340 That's where I got PYTHON_SITE_PKG from. There's also That's going to change though distro to distro... The commands themselves are universal, the result is not. PYTHON_SITE_PKG works but
Using PYTHON_SITE_PKG works for both a system-wide install though, and run-in-place. I just checked in my Trixie VM and PYTHON_SITE_PKG is working as intended (both rip and system-wide install.) Another problem is accounting for Fedora. PYTHON_SITE_PKG works there too but the hard-codes need to be fixed with a simple one-linger, not some 10 line script from autotools. Luckily, this works:
Now I need to fix the other hard-codes but the upcoming code might be wrong.. |
9fc2a6c to
b9b3f0a
Compare
Fixes the need for PYTHONPATH on Gentoo Signed-off-by: Alec Ari <neotheuser@ymail.com>
b9b3f0a to
6146ae4
Compare
|
This might be ready now. Not 100% certain. |
Fixes the need for PYTHONPATH on Gentoo