Skip to content

{bio} [goolf-1.4.10] MM-align (REVIEW) #1428

Merged
boegel merged 15 commits intoeasybuilders:developfrom
pescobar:MM-align
Jun 1, 2016
Merged

{bio} [goolf-1.4.10] MM-align (REVIEW) #1428
boegel merged 15 commits intoeasybuilders:developfrom
pescobar:MM-align

Conversation

@pescobar
Copy link
Copy Markdown
Member

No description provided.

@hpcugentbot
Copy link
Copy Markdown

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

or: run an md5sum or sha1sum, take your preferred number of digits (github likes 7) and use that.
strange? it is a way to deal with developers who litter the digital world with non-provenanced codes.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

or, make the above a -versionsuffix and use a date as a version (probably more broad scope, covers more needs)

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.

+1, let's make this a datestamp

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.

http://zhanglab.ccmb.med.umich.edu/MM-align/ mentions that the last update was 20130815, so use that?

@pescobar
Copy link
Copy Markdown
Member Author

Test report by @pescobar
SUCCESS
Build succeeded for 1 out of 1 (2 easyconfigs in this PR)
login18.cluster.bc2.ch - Linux centos 6.5, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.6.6
See https://gist.github.com/c0e29d1c11ced81316a6b0617d1a9016 for a full test report.

@boegel boegel added this to the v2.x milestone May 29, 2016
@boegel
Copy link
Copy Markdown
Member

boegel commented May 29, 2016

@pescobar wrong easyconfig filename, should be MM-align-1.0-goolf-1.4.10-20130815.eb'

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 1, 2016

@pescobar please remove the faulty MM-align-1.0-goolf-1.4.10.eb?

easyblock = 'CmdCp'

name = 'MM-align'
version = '1.0' # no version info provided. I decide it's 1.0
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.

@pescobar: I would replace this with 20130815 rather than picking a random version number (and drop the versionsuffix)

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Jun 1, 2016

Test report by @pescobar
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
login18.cluster.bc2.ch - Linux centos 6.5, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.6.6
See https://gist.github.com/346ac88bcc77ab9ad139a7d00cdf346e for a full test report.

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Jun 1, 2016

Test report by @pescobar
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
login18.cluster.bc2.ch - Linux centos 6.5, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.6.6
See https://gist.github.com/7d63e5258721988a7225134938561257 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 1, 2016

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2160.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/d94666e6860bd50010edd60df2d83dd9 for a full test report.

@boegel boegel modified the milestones: v2.9.0, v2.x Jun 1, 2016
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 1, 2016

lgtm

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.

4 participants