Skip to content

Connectome work bench#3411

Merged
boegel merged 6 commits intoeasybuilders:developfrom
rtripath89:Connectome_WorkBench
Jan 7, 2017
Merged

Connectome work bench#3411
boegel merged 6 commits intoeasybuilders:developfrom
rtripath89:Connectome_WorkBench

Conversation

@rtripath89
Copy link
Copy Markdown
Contributor

No description provided.

@boegel boegel added this to the 3.1.0 milestone Dec 28, 2016
name = 'ConnectomeWorkbench'
version = '1.2.2'

homepage = 'http://www.humanconnectome.org/software/get-connectome-workbench.html'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

sources = ['workbench-rh_linux64-v%(version)s.zip']

modextrapaths = {'PATH': 'bin_rh_linux64', 'LIBRARY_PATH': 'libs_rh_linux64',
'LD_LIBRARY_PATH': 'libs_rh_linux64'}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please fix formatting:

modextrapaths = {
    'LD_LIBRARY_PATH': 'libs_rh_linux64',
    'LIBRARY_PATH': 'libs_rh_linux64',
    'PATH': 'bin_rh_linux64',
}

@rtripath89
Copy link
Copy Markdown
Contributor Author

@boegel Updated the suggested changes.

modextrapaths = {
'PATH': 'bin_rh_linux64',
'LIBRARY_PATH': 'libs_rh_linux64',
'LD_LIBRARY_PATH': 'libs_rh_linux64'}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@rtripath89 please move the } to the next line:

modextrapaths = {
    ...
    'PATH': 'bin_rh_linux64',
}

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2017

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2017

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 6, 2017

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 7, 2017

Going in, thanks @rtripath89!

@boegel boegel merged commit f9c1982 into easybuilders:develop Jan 7, 2017
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.

2 participants