Skip to content

fix version check + sanity check in Geant4 easyblock + style fixes#517

Merged
boegel merged 4 commits intoeasybuilders:developfrom
boegel:geant4_94
Dec 17, 2014
Merged

fix version check + sanity check in Geant4 easyblock + style fixes#517
boegel merged 4 commits intoeasybuilders:developfrom
boegel:geant4_94

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Dec 3, 2014

No description provided.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Dec 3, 2014

@wpoely86: please review?

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.

it's the same for both cases, so why not put it directly here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

no, it's G vs g

@hpcugentbot
Copy link
Copy Markdown

Test PASSed.

1 similar comment
@hpcugentbot
Copy link
Copy Markdown

Test PASSed.

@hpcugentbot
Copy link
Copy Markdown

Test PASSed.

@hpcugentbot
Copy link
Copy Markdown

Test PASSed.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Dec 17, 2014

Jenkins: test this please

@hpcugentbot
Copy link
Copy Markdown

Test PASSed.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Dec 17, 2014

tested with easybuilders/easybuild-easyconfigs#1228, going in, thanks for the review @wpoely86!

boegel added a commit that referenced this pull request Dec 17, 2014
fix version check + sanity check in Geant4 easyblock + style fixes
@boegel boegel merged commit 51e59c5 into easybuilders:develop Dec 17, 2014
@boegel boegel deleted the geant4_94 branch December 17, 2014 14:30
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.

3 participants