Skip to content

{bio}[dummy/] IMOD v4.7.15#3347

Merged
boegel merged 2 commits intoeasybuilders:developfrom
pescobar:20160714112315_new_pr_IMOD4715
Aug 17, 2016
Merged

{bio}[dummy/] IMOD v4.7.15#3347
boegel merged 2 commits intoeasybuilders:developfrom
pescobar:20160714112315_new_pr_IMOD4715

Conversation

@pescobar
Copy link
Copy Markdown
Member

@pescobar pescobar commented Jul 14, 2016

(created using eb --new-pr)

boegel
boegel reviewed Aug 10, 2016
View reviewed changes
Comment thread easybuild/easyconfigs/i/IMOD/IMOD-4.7.15_RHEL6-64_CUDA6.0.eb
}

sanity_check_paths = {
'files': ['IMOD/bin/imod'],
Copy link
Copy Markdown
Member

@boegel boegel Aug 10, 2016

Choose a reason for hiding this comment

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

imod will not be available via $PATH this way...

we should add IMOD/bin to $PATH via modextrapaths?

Copy link
Copy Markdown
Member Author

@pescobar pescobar Aug 17, 2016

Choose a reason for hiding this comment

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

the easyconfig includes this
modloadmsg = """Bash users run: 'source $EBROOTIMOD/IMOD-linux.sh' csh users run: 'source $EBROOTIMOD/IMOD-linux.csh'"""

when source this file the PATH and other required variables will be defined

Copy link
Copy Markdown
Member

@boegel boegel Aug 17, 2016

Choose a reason for hiding this comment

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

OK, missed that, thanks @pescobar

@boegel boegel added this to the v2.9.0 milestone Aug 10, 2016
@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 10, 2016

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2006.delcatty.os - Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/8aea0a2242102f9cfd7324fe9c9926d4 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 17, 2016

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2159.delcatty.os - Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/b5531637834ea69ec275b9529d7f6f15 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 17, 2016

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 17, 2016

Going in, thanks @pescobar!

@boegel boegel merged commit fbac409 into easybuilders:develop Aug 17, 2016
This was referenced Aug 26, 2016
Merged
Merged
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

v2.9.0

Development

Successfully merging this pull request may close these issues.

2 participants

@pescobar @boegel