Skip to content

pigz easyconfigs for goolf-1.4.10 and ictce-6.2.5#1036

Merged
boegel merged 3 commits intoeasybuilders:developfrom
pescobar:pigz
Mar 24, 2015
Merged

pigz easyconfigs for goolf-1.4.10 and ictce-6.2.5#1036
boegel merged 3 commits intoeasybuilders:developfrom
pescobar:pigz

Conversation

@pescobar
Copy link
Copy Markdown
Member

No description provided.

@fgeorgatos
Copy link
Copy Markdown
Contributor

fyi. this works in favor of this issue as well:
easybuilders/easybuild-easyblocks#82
thanks!

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.

this IS nitpicking: some of us leave 1 space indent in description, some others 2 spaces; I'm probably impartial to either case, yet let's all follow one style to keep things alike - and have less things to bitch over. :-P
Except saving 1 char, is there an advantage to the above?

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.

I always do 1 space indent in description and IIRC I think you suggested me that indentation in my first commits :)

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.

yeap, you are right, that was me about the indentation. just keep doing what you do ;-)

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 28, 2014

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 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/bf5f03cff2c52f1914e0 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 28, 2014

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 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/8df6797f14d670355f9d for a full test report.

@fgeorgatos
Copy link
Copy Markdown
Contributor

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

@fgeorgatos
Copy link
Copy Markdown
Contributor

Hi, please ignore my test of FAILED status of Feb15, it was just caused by a missing icc source (it won't come automatically unless someone feeds it by hand).

It would be nice to see this PR merged in, otherwise duplication of work is unavoidable. Point in case:

https://github.com/emi80/easyconfigs/blob/master/p/pigz/pigz-2.3.1-goolf-1.4.10-no-OFED.eb
(!)

makeopts -> buildopts
@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/2676/
Test PASSed.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 24, 2015

Test report by @boegel
FAILED
Build succeeded for 1 out of 2 (2 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/9e5bd045ca85f9e17c11 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 24, 2015

ignore the failed test, screwup on my end, retrying

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 24, 2015

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 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/356a89d3ae20ebe674fb for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 24, 2015

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

boegel added a commit that referenced this pull request Mar 24, 2015
pigz easyconfigs for goolf-1.4.10 and ictce-6.2.5
@boegel boegel merged commit 64868df 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.

4 participants