Skip to content

sickle-1.210-goolf-1.4.10.eb#651

Merged
boegel merged 6 commits intoeasybuilders:developfrom
pescobar:sickle
Feb 14, 2014
Merged

sickle-1.210-goolf-1.4.10.eb#651
boegel merged 6 commits intoeasybuilders:developfrom
pescobar:sickle

Conversation

@pescobar
Copy link
Copy Markdown
Member

@pescobar pescobar commented Jan 7, 2014

No description provided.

@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

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.

if I recall correctly, the 7-digit truncation of a git commit ID works equally well with github and helps readability;

furthermore, if you make it with a versionsuffix it may really help to generalize the easyconfig (think of customized --try-* builds)

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Jan 8, 2014

@fgeorgatos I am not sure how you want the version suffix.

Is ok like this? https://gist.github.com/pescobar/8323332

This way you can run something like "eb /tmp/sickle-1.210-goolf-1.4.10.eb --try-software-version=1.210 --try-amend=versionsuffix=bb996a2 -f"

@fgeorgatos
Copy link
Copy Markdown
Contributor

@pescobar : at this moment the gist url does not appear to work, but from the context I can say yes, that's the point

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.

nitpicking style remark: spaces around =

@pescobar
Copy link
Copy Markdown
Member Author

@boegel better now?

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 14, 2014

@pescobar: perfect, going in! Thanks!

boegel added a commit that referenced this pull request Feb 14, 2014
sickle-1.210-goolf-1.4.10.eb
@boegel boegel merged commit 7e55380 into easybuilders:develop Feb 14, 2014
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