Skip to content

enhance generic RPackage easyblock to support installing extensions in a separate prefix#551

Merged
boegel merged 6 commits intoeasybuilders:developfrom
boegel:RPackage_enhanced
Feb 6, 2015
Merged

enhance generic RPackage easyblock to support installing extensions in a separate prefix#551
boegel merged 6 commits intoeasybuilders:developfrom
boegel:RPackage_enhanced

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Feb 5, 2015

This should also resolve issue #1326 where R packages are being installed under $HOME rather than in the R installation prefix...

@wpoely86, @rjeschmi: please review/test?

@hpcugentbot
Copy link
Copy Markdown

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

@hpcugentbot
Copy link
Copy Markdown

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

@rjeschmi
Copy link
Copy Markdown
Contributor

rjeschmi commented Feb 6, 2015

Seems like a bug

https://gist.github.com/rjeschmi/3cb1c213000eeafbb0df

== FAILED: Installation ended unsuccessfully (build directory: /software/experimental/build/R/3.1.2/foss-2015a): build failed (first 300 chars): EasyBuild crashed with an error (at easybuild/tools/run.py:382 in parse_cmd_output): cmd "R -q --no-save" exited with exitcode 1 and output:

    options(repos=c(CRAN="http://www.freestatistics.org/cran"))


    install.packages("base", /software/experimental/softwa

@rjeschmi
Copy link
Copy Markdown
Contributor

rjeschmi commented Feb 6, 2015

I think it is missing quotes around the lib path, will add

@hpcugentbot
Copy link
Copy Markdown

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

@rjeschmi
Copy link
Copy Markdown
Contributor

rjeschmi commented Feb 6, 2015

It looks like the build completed successfully after I patched

https://gist.github.com/rjeschmi/84ff829c04269e0f17fc

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 6, 2015

@rjeschmi: great, thanks for the feedback!

@hpcugentbot
Copy link
Copy Markdown

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

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 6, 2015

Also retested this myself together with R/3.1.2-foss-2015a from easybuilders/easybuild-easyconfigs#1357, working great.

It's also serving the purpose of the PoC PR easybuilders/easybuild-easyconfigs#1384.

Thanks a lot for testing/reviewing this @rjeschmi, and for the fix you provided!

Going in!

boegel added a commit that referenced this pull request Feb 6, 2015
enhance generic RPackage easyblock to support installing extensions in a separate prefix
@boegel boegel merged commit 1dbe40b into easybuilders:develop Feb 6, 2015
@boegel boegel deleted the RPackage_enhanced branch February 6, 2015 22:37
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.

3 participants