Skip to content

pkg-config include as builddependencies#2827

Merged
boegel merged 9 commits intoeasybuilders:developfrom
jrbosch:libxcb
Apr 6, 2016
Merged

pkg-config include as builddependencies#2827
boegel merged 9 commits intoeasybuilders:developfrom
jrbosch:libxcb

Conversation

@jrbosch
Copy link
Copy Markdown
Contributor

@jrbosch jrbosch commented Apr 5, 2016

If pkg-config is not installed in the system. Some packages returns error in the configure step. To avoid this is necessary include pkg-config as builddependencies

@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

@hajgato
Copy link
Copy Markdown
Collaborator

hajgato commented Apr 6, 2016

@jrbosch check PR #2743 and related PRs.

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 6, 2016

Jenkins: ok to test

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 6, 2016

@jrbosch Thank you for your contribution!

As @hajgato pointed out, there is another open pull request that partially overlaps with this, see #2743.

I'll look into merging that PR first, since there has been quite a bit on cleanup in these X11-related easyconfigs lately by @hajgato, w.r.t. build dependencies, etc.

I will get back to this PR once #2743 is merged in; I can help you with syncing up your branch to get this merge-ready afterwards. Please don't hesitate to ping for an update in case you don't hear back.

@boegel boegel added this to the v2.8.0 milestone Apr 6, 2016
@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7342/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 6, 2016

@jrbosch Please see jrbosch#1 which resolves the merge conflicts that now occur after merging #2743.

sync with develop & resolve conflicts
source_urls = ['http://xorg.freedesktop.org/archive/individual/lib/']

builddependencies = [
('pkg-config', '0.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.

@jrbosch I managed to introduce a syntax error here, leading to this error:

TypeError: 'tuple' object is not callable

The problem is that there's a comma (,) missing at the end of this line, so please change this to:

    ('pkg-config', '0.29'),

@jrbosch jrbosch changed the title {lib,devel,lib,vis,vis,devel}[foss/2016a] libxcb 1.11.1, xproto 7.0.28, libpthread-stubs 0.3, libXau 1.0.8, libXdmcp-1.1.2, pkg-config-0.29 (REVIEW) pkg-config include as builddependencies Apr 6, 2016
@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7349/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 6, 2016

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 6, 2016

Test report by @boegel
SUCCESS
Build succeeded for 4 out of 4 (4 easyconfigs in this PR)
node2159.delcatty.os - Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/d403251561fd30d4ed0f2422154b905f for a full test report.

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7350/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 6, 2016

Good to go, thanks @jrbosch, and congratulations on your first contribution to EasyBuild! Keep them coming!

@boegel boegel merged commit 5c43ecf into easybuilders:develop Apr 6, 2016
@jrbosch jrbosch deleted the libxcb branch April 6, 2016 20:59
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