Use env wherever `prebuildopts is used to set environmental variables.#2811
Conversation
|
Automatic reply from Jenkins: Can I test this? |
|
Jenkins: ok to test |
|
@riccardomurri we have a single instance of this for let's fix that in here too? |
|
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7262/console for more details). This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
|
@riccardomurri merging #2807 resulted in some conflicts here due to lines in context being modified, please resync with |
48c8478 to
ac6e72b
Compare
|
(Kenneth Hoste, Thu, Mar 31, 2016 at 09:06:10AM -0700:)
Done, but I had to do a |
| #preconfigopts = ' PKG_CONFIG_PATH="$PKG_CONFIG_PATH:/usr/lib64/pkgconfig/:/usr/share/pkgconfig" ' | ||
|
|
||
| prebuildopts = 'CPATH="$EBROOTLIBDRM/include/libdrm" ' | ||
| prebuildopts = 'env CPATH="$EBROOTLIBDRM/include/libdrm" ' |
There was a problem hiding this comment.
eek, I don't think we should be hard redefining $CPATH and totally ignore the current value here...
@riccardomurri fixed with riccardomurri#5
|
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7283/console for more details). This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
don't hard overwrite $CPATH in prebuildopts in Mesa easyconfig files
|
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7295/console for more details). This pull request is now ready for review/testing. Please try and find someone who can tackle this; contact @boegel if you're not sure what to do. |
|
Test report by @boegel |
|
Test report by @boegel |
|
The issue with motif 2.2.4 with Going in, thanks @riccardomurri! |
As requested by @boegel in PR #2807