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

PyHSPlasma picks up Python 3, installs to Python 2 path #47

Closed
diafero opened this issue Oct 20, 2014 · 2 comments
Closed

PyHSPlasma picks up Python 3, installs to Python 2 path #47

diafero opened this issue Oct 20, 2014 · 2 comments

Comments

@diafero
Copy link
Member

diafero commented Oct 20, 2014

It seems that the Python module is compiled against Python 3, but installed to the Python 2 path:

$ nm /usr/lib/python2.7/dist-packages/PyHSPlasma.so -D|fgrep PyHSPlasma
00000000000cde00 T PyInit_PyHSPlasma

If I understand the source properly, this symbol should only exist for Python 3 modules. Loading it in Python 2 gives the following error:

ImportError: dynamic module does not define init function (initPyHSPlasma)

I'm not sure what changed to break this. It's certainly not in libhsplasma, as the breakage was introduced after a re-build yesterday. But probably the Python stuff in cmake is should somehow make sure that the versions match. (And Python 2 is, for now, the version I'd want, but it would be okay if I would have to configure that explicitly.)

@diafero
Copy link
Member Author

diafero commented Oct 20, 2014

Indeed cmake confirms my suspicion:

-- Found PythonLibs: /usr/lib/x86_64-linux-gnu/libpython3.4m.so (found version "3.4.2") 
-- Found PythonInterp: /usr/bin/python (found version "2.7.8")

@diafero
Copy link
Member Author

diafero commented Jul 12, 2015

This has been fixed a while ago by the merge request referenced above.

@diafero diafero closed this as completed Jul 12, 2015
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

1 participant