Skip to content

{bio}[goolf/1.4.10] Minimac 20140110 (REVIEW)#815

Merged
boegel merged 4 commits intoeasybuilders:developfrom
pescobar:Minimac
Mar 24, 2015
Merged

{bio}[goolf/1.4.10] Minimac 20140110 (REVIEW)#815
boegel merged 4 commits intoeasybuilders:developfrom
pescobar:Minimac

Conversation

@pescobar
Copy link
Copy Markdown
Member

No description provided.

@fgeorgatos
Copy link
Copy Markdown
Contributor

SUCCESS
Build succeeded for 1 out of 1
Linux debian 6.0.9, Intel(R) Xeon(R) CPU L5640 @ 2.27GHz, Python 2.6.6
See https://gist.github.com/78dd74eaf6c83a146ce6 for a full test report.

@fgeorgatos
Copy link
Copy Markdown
Contributor

looks good - and PR worked, too!

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.

btw. the location of this comma is a bit weird; is this really the intended style?

@fgeorgatos
Copy link
Copy Markdown
Contributor

Test report by @fgeorgatos
FAILED
Build succeeded for 0 out of 1
Linux debian 6.0.10, Intel(R) Xeon(R) CPU L5640 @ 2.27GHz, Python 2.6.6
See https://gist.github.com/28db2609298a8cb1ab3e for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 21, 2015

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

@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/2642/
Test FAILed.

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.

why beta?

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.

Hi folks,

I try to enter into Pablo's mind and I think he would have liked to propose instead:

version = '2013.7.17'
versionsuffix = '-a8011a4'

More generally, for nasty version-less applications we should adopt an ad-hoc format like:

version = '20130717'
versionsuffix = '-a8011a4' # or any other hash for that matter

The point is:

  • Make it possible to sort sanely, by temporal aspects / release time
  • Confine the version/naming sufficiently to make the builds truly unique.

Your takes?

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.

makes sense, yeah

We should be using the datestamp in the source tarball name too, but that breaks auto-downloading.

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.

on second thought, just specifying a datestamp as 'version' as using that in the source name should do it, no need to add a hash as a version suffix (the datestamp serves that purpose, together with the specified MD5sum being checked)

@rjeschmi
Copy link
Copy Markdown
Contributor

Test report by @rjeschmi
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux centos 6.6, Intel(R) Xeon(R) CPU E5440 @ 2.83GHz, Python 2.6.6
See https://gist.github.com/4c0db7652130ac4b4e5a for a full test report.

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.

include datestamp/version in source tarball name (breaks auto-downloading, but that's not a big issue imho)

# download minimac.src.tgz from http://www.sph.umich.edu/csg/cfuchsb and rename
sources = ['%(namelower)s-%(version)s.src.tgz']

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 24, 2015

@pescobar: please review/merge pescobar#13, so we can get this one merged in...

@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/2666/
Test PASSed.

@boegel boegel changed the title Minimac-2013.7.17-goolf-1.4.10.eb {bio}[goolf/1.4.10] Minimac 20140110 (REVIEW) Mar 24, 2015
@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 24, 2015

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 24, 2015

(finally) going in, thanks @pescobar, and @fgeorgatos, @rjeschmi for testing/reviewing

boegel added a commit that referenced this pull request Mar 24, 2015
{bio}[goolf/1.4.10] Minimac 20140110 (REVIEW)
@boegel boegel merged commit 30422dd into easybuilders:develop Mar 24, 2015
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.

5 participants