Skip to content

{geo}[intel/2017b] PCRaster 4.1.0 (REVIEW)#5386

Merged
boegel merged 1 commit intoeasybuilders:developfrom
hajgato:pcraster410
Nov 22, 2017
Merged

{geo}[intel/2017b] PCRaster 4.1.0 (REVIEW)#5386
boegel merged 1 commit intoeasybuilders:developfrom
hajgato:pcraster410

Conversation

@hajgato
Copy link
Copy Markdown
Collaborator

@hajgato hajgato commented Nov 22, 2017

No description provided.

boegel
boegel approved these changes Nov 22, 2017
View reviewed changes
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 22, 2017

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2452.golett.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/e902aa58beaeff91b1bbabec38b86332 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 22, 2017

@hajgato Any particular reason you're using the binary release rather than building from source?

@boegel boegel added this to the 3.5.0 milestone Nov 22, 2017
@boegel boegel added the new label Nov 22, 2017
@hajgato
Copy link
Copy Markdown
Collaborator Author

hajgato commented Nov 22, 2017

@boegel I did not find the source.

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 22, 2017

@hajgato https://github.com/pcraster/pcraster/ (granted, there are no proper releases, it seems...)

@hajgato
Copy link
Copy Markdown
Collaborator Author

hajgato commented Nov 22, 2017

@boegel Suggestions? User asked specifically version 4.1.0.

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 22, 2017

@hajgato Hmm, then I guess the binary version is your best option, since there's a total lack of tags/releases in the source repository...

Maybe open an issue to ask them about that?

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 22, 2017

Going in, thanks @hajgato!

@boegel boegel merged commit 40000b1 into easybuilders:develop Nov 22, 2017
@hajgato hajgato deleted the pcraster410 branch February 12, 2018 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

new

Projects

None yet

Milestone

3.5.0

Development

Successfully merging this pull request may close these issues.

2 participants

@hajgato @boegel