Skip to content

also set $PYTHONPATH before installing Python pkgs in tmp dir in test_step#591

Merged
boegel merged 1 commit intoeasybuilders:developfrom
boegel:PYTHONPATH_before_test_install
Apr 9, 2015
Merged

also set $PYTHONPATH before installing Python pkgs in tmp dir in test_step#591
boegel merged 1 commit intoeasybuilders:developfrom
boegel:PYTHONPATH_before_test_install

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Apr 9, 2015

I'm somewhat surprised this hasn't popped up before... ran into this when test-installing an update for netcdf4-python...

error: bad install directory or PYTHONPATH
You are attempting to install a package to a directory that is not
on PYTHONPATH and which Python does not read ".pth" files from.
The installation directory you specified (via --install-dir, --prefix, or
the distutils default setting) was:

    /tmp/eb-5o9D62/tmpCohFbi/lib/python2.7/site-packages/                                                                                                                       

and your PYTHONPATH environment variable currently contains: ...

@wpoely86: please review?

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Apr 9, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyblocks-pr-builder/908/
Test PASSed.

wpoely86
wpoely86 reviewed Apr 9, 2015
View reviewed changes
Comment thread easybuild/easyblocks/generic/pythonpackage.py
Copy link
Copy Markdown
Member

@wpoely86 wpoely86 Apr 9, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be: %s %s python ?

Copy link
Copy Markdown
Member

@wpoely86 wpoely86 Apr 9, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a no, ignore

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Apr 9, 2015

Thanks for the review @wpoely86!

boegel added a commit that referenced this pull request Apr 9, 2015
@boegel
Merge pull request #591 from boegel/PYTHONPATH_before_test_install
06a5bf3 
also set $PYTHONPATH before installing Python pkgs in tmp dir in test_step
@boegel boegel merged commit 06a5bf3 into easybuilders:develop Apr 9, 2015
@boegel boegel deleted the PYTHONPATH_before_test_install branch April 9, 2015 14:36
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

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@boegel @hpcugentbot @wpoely86