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

Fix to better support python installs where prefix != exec_prefix. #636

Closed

Conversation

shidesthewise
Copy link

Patch version from Hans Meine from the duistutils mail list.
https://mail.python.org/pipermail/distutils-sig/2008-April/009336.html
http://mail.python.org/pipermail/distutils-sig/attachments/20080414/16fb135e/attachment-0001.diff

Add copy_inc_files function to fix issue of lost pyconfig.h.

When sys.exec_prefix and sys.prefix differ the pyconfig.h file
is in $(exec_prefix)/include/python$(version) and that file is
missing in the virtualenv because env/include/python$(version)
is a symlink to $(real_prefix)/include/python$(version).
If --always-copy is used then the two include directories merge.
I added copy_inc_files to symlink (or copy) the individual contents
of prefix include and exec_prefix include into env/include.

https://mail.python.org/pipermail/distutils-sig/2008-April/009336.html
http://mail.python.org/pipermail/distutils-sig/attachments/20080414/16fb135e/attachment-0001.diff

Add copy_inc_files function to fix issue of lost pyconfig.h.

When sys.exec_prefix and sys.prefix differ the pyconfig.h file
is in $(exec_prefix)/include/python$(version) and that file is
missing in the virtualenv because env/include/python$(version)
is a symlink to $(real_prefix)/include/python$(version).
If --always-copy is used then the two include directories merge.
I added copy_inc_files to symlink (or copy) the individual contents
of prefix include and exec_prefix include into env/include.
Skip copying required file if exec_prefix != prefix and
the file is sourced from exec_prefix.  Let the
copying/linking of exec_dir in install_python handle
these files.
@dstufft dstufft mentioned this pull request Jan 5, 2015
5 tasks
@shidesthewise
Copy link
Author

Because this is old and based on 1.11 and virtualenv is now in a 13.X series, I'll re-submit.

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

Successfully merging this pull request may close these issues.

2 participants