Skip to content

BUG: add missing libpng dependency to g2lib-1.4.0 for intel-2017a#5196

Merged
boegel merged 2 commits intoeasybuilders:developfrom
andreas-h:ncl_fix
Oct 11, 2017
Merged

BUG: add missing libpng dependency to g2lib-1.4.0 for intel-2017a#5196
boegel merged 2 commits intoeasybuilders:developfrom
andreas-h:ncl_fix

Conversation

@andreas-h
Copy link
Copy Markdown
Contributor

When trying to build g2lib-1.4.0-intel-2017a, I got an error about png.h not being found. With this fix, the build succeeds.

@andreas-h
Copy link
Copy Markdown
Contributor Author

The same was true for g2clib


dependencies = [('JasPer', '1.900.1')]
dependencies = [('JasPer', '1.900.1'),
('libpng', '1.6.29'),]
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.

@andreas-h Please reformat to match style used in other easyconfigs:

dependencies = [
    ('JasPer', '1.900.1'),
    ('libpng', '1.6.29'),
]

@boegel boegel added this to the 3.4.1 milestone Oct 11, 2017
@easybuilders easybuilders deleted a comment from boegelbot Oct 11, 2017
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 Oct 11, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2670.swalot.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/4ee6ea46501fa64918ba6276f7436219 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 11, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2127.delcatty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/73e7b777d097e1b8ef910830619034cb for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 11, 2017

Going in, thanks @andreas-h!

@boegel boegel merged commit 01a3526 into easybuilders:develop Oct 11, 2017
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.

2 participants