Skip to content

hardcode Python lib path, since setuptools/distribute hardcode it#184

Merged
boegel merged 2 commits intoeasybuilders:developfrom
boegel:det_pylibdir
Apr 29, 2013
Merged

hardcode Python lib path, since setuptools/distribute hardcode it#184
boegel merged 2 commits intoeasybuilders:developfrom
boegel:det_pylibdir

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Apr 29, 2013

lib/python-2.x/site-packages is hardcoded by setuptools and distribute (see below), so we shouldn't be relying on distutils.sysconfig.get_python_lib because it doesn't work as we want/expect on various OSs

setuptools-0.6c11 $ sed -n "1107,1114p" setuptools/command/easy_install.py

    INSTALL_SCHEMES = dict(
        posix = dict(
            install_dir = '$base/lib/python$py_version_short/site-packages',
            script_dir  = '$base/bin',
        ),
    )

and

distribute-0.6.35 $ sed -n "1322,1327p" setuptools/command/easy_install.py
    INSTALL_SCHEMES = dict(
        posix = dict(
            install_dir = '$base/lib/python$py_version_short/site-packages',
            script_dir  = '$base/bin',
        ),
    )

This patch should fix easybuilders/easybuild-framework#595 and easybuilders/easybuild-framework#597.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Apr 29, 2013

Tested this patch by building separate numpy and scipy modules on top of Python, works fine.

@JensTimmerman
Copy link
Copy Markdown

JensTimmerman commented Apr 29, 2013

Error message should not be a question,

Ok otherwise.

boegel added a commit that referenced this pull request Apr 29, 2013
@boegel
Merge pull request #184 from boegel/det_pylibdir
8c518cb 
hardcode Python lib path, since setuptools/distribute hardcode it
@boegel boegel merged commit 8c518cb into easybuilders:develop Apr 29, 2013
@boegel boegel deleted the det_pylibdir branch April 29, 2013 11:17
This was referenced Apr 29, 2013
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v1.4

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @JensTimmerman