Skip to content

easyconfig file for BOINC#436

Merged
boegel merged 9 commits intoeasybuilders:developfrom
wdpypere:boinc
Sep 17, 2013
Merged

easyconfig file for BOINC#436
boegel merged 9 commits intoeasybuilders:developfrom
wdpypere:boinc

Conversation

@wdpypere
Copy link
Copy Markdown
Contributor

recreated pull request targetting develop branch.

@boegel boegel mentioned this pull request Sep 17, 2013
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.

line too long, please break it up like:

description = """BOINC ...
others."""

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.

Better:

files_to_copy = [(['client/boinc', 'client/boinccmd'], 'bin')]

That's exactly why the first element of the tuple is a list...

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.

This is a clear workaround for the fact that the MakeCp easyblock doesn't have a configure step (yes, I know I suggested that to you).

Let me construct a ConfigureMakeCp easyblock, so you can use that instead, and adjust the above to:

preconfigopts = './_autosetup &&'
configopts = '--disable-server --disable-manager --enable-client'

use MakeCp easyblock, enable configure
@boegel
Copy link
Copy Markdown
Member

boegel commented Sep 17, 2013

Merging in, thanks @wdpypere!

boegel added a commit that referenced this pull request Sep 17, 2013
easyconfig file for BOINC
@boegel boegel merged commit 07ddf14 into easybuilders:develop Sep 17, 2013
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