Skip to content

Add cpan minus extension to recent perl easyconfigs#7866

Merged
boegel merged 4 commits intoeasybuilders:developfrom
pescobar:add-cpanm-to-perl
Mar 13, 2019
Merged

Add cpan minus extension to recent perl easyconfigs#7866
boegel merged 4 commits intoeasybuilders:developfrom
pescobar:add-cpanm-to-perl

Conversation

@pescobar
Copy link
Copy Markdown
Member

@pescobar pescobar commented Mar 12, 2019

this extension is quite convenient so users can easily install extra perl libraries to their home folders. e.g.

$> module load Perl
$> cpanm LWP::Protocol::https
$> export PERL5LIB="$HOME/perl5/lib/perl5:$PERL5LIB"
$> export PATH="$HOME/perl5/bin:$PATH"

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Mar 12, 2019

why travis complains if I only updated easyconfigs that are already in master branch?

@akesandgren
Copy link
Copy Markdown
Contributor

akesandgren commented Mar 12, 2019

Because every new PR goes through the verification step in travis, like checking for checksums etc.

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Mar 12, 2019

@pescobar it is probably more convenient to just add the extension to the more recent easyconfigs, ie the ones that already have checksums.

@boegel boegel added this to the 3.x milestone Mar 12, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 12, 2019

@pescobar I would stick to only adding that extension to the most recent Perl easyconfigs, indeed. Versions 5.26.0 and newer would work.

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Mar 13, 2019

I have updated the PR as you suggested and Travis is happy now

@easybuilders easybuilders deleted a comment from boegelbot Mar 13, 2019
boegel
boegel approved these changes Mar 13, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm after visual review

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 13, 2019

Test report by @boegel
SUCCESS
Build succeeded for 8 out of 8 (8 easyconfigs in this PR)
node2002.delcatty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/60cd12fc88623f3c6a18960a8977f07e for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 13, 2019

Test report by @boegel
SUCCESS
Build succeeded for 8 out of 8 (8 easyconfigs in this PR)
node2573.golett.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/0e0f9e6a872567540531bcacd40d68c8 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 13, 2019

Going in, thanks @pescobar!

@boegel boegel merged commit 0600b2c into easybuilders:develop Mar 13, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 13, 2019

Test report by @boegel
SUCCESS
Build succeeded for 8 out of 8 (8 easyconfigs in this PR)
node3108.skitty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/2dabfee75c7da848386ca99d26e4834c for a full test report.

@boegel boegel modified the milestones: 3.x, next release (3.8.2) Mar 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

enhancement

Projects

None yet

Milestone

3.9.0

Development

Successfully merging this pull request may close these issues.

4 participants

@pescobar @akesandgren @smoors @boegel