add missing libX dependencies for cairo

[boegel]

When looking into adding support for Glade, I ran into missing dependencies for cairo~~, and some misspecified build deps in X11 libraries that should actually be runtime dependencies (as specified in the pkg-config files for these libraries).~~

The latter was changed in the latest 2016 easyconfigs for the X11 libraries done by @hajgato (cfr. #2549, #2563, #2614)

[boegel]

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in this PR)
node2151.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/7b330d528ba95ff5aeae for a full test report.

[boegel]

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in this PR)
node2668.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/4164b01764d2d23be203 for a full test report.

[hpcugentbot]

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/6786/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.

[hajgato]

protos are always headers and nothing else, so I suggest them to use only as builddeps, and never deps. This means that if you need to build something that requires some X11 libs, you might have to include some of the protos as a builddep as well. (not always).

[hajgato]

@boegel: mayve we have to explicitly add libX11 as dependency (despite is automatically loaded by those Xlibs). The shared object is also depends on libX11.so. But it is up to your taste, otherwise it is fine.

[boegel]

@hajgato: makes sense, will do, thanks for the suggestion

[hajgato]

@boegel:
shoot, you might need to add

builddependencies = [
    ('renderproto', '0.11'),
]

[boegel]

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

[boegel]

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2668.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/bd2ac6ff7c66a35c7908 for a full test report.

[boegel]

@hajgato: ok, will add that too

[hajgato]

above dependencies?
extra empty line?

[boegel]

We don't have strict guidelines for this imho..

I think you'll find all combos w.r.t. order and empty line in between or not.

Let's change this consistently once we have an automated check for it.

[hajgato]

OK, then. :)

[hpcugentbot]

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/6787/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.

[hpcugentbot]

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/6788/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.

[hpcugentbot]

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/6789/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]

Thanks for the review @hajgato!