Skip to content

enable pic option in expat easyconfigs#1562

Merged
boegel merged 1 commit intoeasybuilders:developfrom
boegel:expat_fPIC
Apr 22, 2015
Merged

enable pic option in expat easyconfigs#1562
boegel merged 1 commit intoeasybuilders:developfrom
boegel:expat_fPIC

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Apr 22, 2015

No description provided.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Apr 22, 2015

@wpoely86: please review?

@wpoely86
Copy link
Copy Markdown
Member

wpoely86 commented Apr 22, 2015

looks fine

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Apr 22, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2840/
Test PASSed.

@boegel boegel mentioned this pull request Apr 22, 2015
Merged
@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Apr 22, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2842/
Test FAILed.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Apr 22, 2015

Test report by @boegel
FAILED
Build succeeded for 17 out of 18 (18 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/b41ced5565b042f0f39f for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Apr 22, 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/6fe28b644e1b4bf6eaa4 for a full test report.

@fgeorgatos
Copy link
Copy Markdown
Contributor

fgeorgatos commented Apr 22, 2015

just for the record: what triggered the change?

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Apr 22, 2015

@fgeorgatos: #1529

@hpcugentbot
Copy link
Copy Markdown

hpcugentbot commented Apr 22, 2015

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/2844/
Test PASSed.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Apr 22, 2015

going in, just a minor hickup in the test report, but all is well, thanks for the review @wpoely86!

boegel added a commit that referenced this pull request Apr 22, 2015
@boegel
Merge pull request #1562 from boegel/expat_fPIC
828c2d3 
enable pic option in expat easyconfigs
@boegel boegel merged commit 828c2d3 into easybuilders:develop Apr 22, 2015
@boegel boegel deleted the expat_fPIC branch April 22, 2015 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@boegel @wpoely86 @hpcugentbot @fgeorgatos