add xkeyboard-config component in X11 bundle + intltool as required build dep (REVIEW) by boegel · Pull Request #5066 · easybuilders/easybuild-easyconfigs

boegel · 2017-08-28T08:12:24Z

(WIP because this change should be made consistently also in the other X11 easyconfigs; the intltool build dep, which in turns required Perl, is a bit of a PITA though...)

…uild dep

jhein32 · 2017-08-28T08:23:31Z

@boegel
Hi Kenneth,

This is work highly appreciated. It seems you are "upgrading" the old-ish x11 config here.

I assume this will be needed for the newer x11 as well. What is the plan for doing this?

If this is merged into devel, I could have a try at adding this to the more modern x11.

Joachim

boegel · 2017-08-28T08:38:17Z

@jhein32 As I mentioned in the description, this is WIP, mainly to see if this fixes the problem you reported on the mailing list (https://lists.ugent.be/wws/arc/easybuild/2017-08/msg00042.html).

If it does, I'm happy to look into applying this change to the other (recent) X11 easyconfigs too.

I see this as a bug fix, not an upgrade; as your problem shows, the X11 bundle as it is now is incomplete.

jhein32 · 2017-08-28T09:27:24Z

@boegel

thanks. Sorry, I overlooked the first box on github. It is currently building Perl. I report back tomorrow.

JackPerdue · 2017-08-28T12:52:41Z

FWIW, PR #4979 and #4980 have some updated version numbers....

jhein32 · 2017-08-30T18:44:20Z

@boegel I got feedback from the user. They claim the qt5 I built on top of this X11 is now working. On 28 Aug 2017, at 10:38, Kenneth Hoste <notifications@github.com<mailto:notifications@github.com>> wrote: @jhein32<https://github.com/jhein32> As I mentioned in the description, this is WIP, mainly to see if this fixes the problem you reported on the mailing list (https://lists.ugent.be/wws/arc/easybuild/2017-08/msg00042.html). If it does, I'm happy to look into applying this change to the other (recent) X11 easyconfigs too. I see this as a bug fix, not an upgrade; as your problem shows, the X11 bundle as it is now is incomplete. - You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#5066 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/APK9dOCIrjrsMyw6YpOwLikpjioU_dg8ks5scnx6gaJpZM4PENiO>.

…tltool as build dep

…nce sanity check

boegel · 2017-09-07T20:14:02Z

Test report by @boegel
FAILED
Build succeeded for 46 out of 49 (19 easyconfigs in this PR)
node2415.golett.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/fe90516e345bf9c93ea37d98f86d0602 for a full test report.

boegel · 2017-10-12T15:56:31Z

Test report by @boegel
FAILED
Build succeeded for 18 out of 19 (19 easyconfigs in this PR)
node2075.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/2cd3c23c2ecdcc645f227dedee41586b for a full test report.

edit: segfault in failing build caused by known bug in one of the Intel libraries which causes problems with glibc in CentOS 7, fixed by copying that library from a more recent version of Intel compilers, see also https://bugzilla.redhat.com/show_bug.cgi?id=1377895

boegel · 2017-10-12T23:05:02Z

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

vanzod · 2017-10-12T23:47:19Z

Test report by @vanzod
SUCCESS
Build succeeded for 12 out of 12 (3 easyconfigs in this PR)
cermis - Linux debian 8.8, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.7.9
See https://gist.github.com/7c73b326146736130e544b84a4e66319 for a full test report.

migueldiascosta · 2017-10-13T07:18:47Z

+        'source_tmpl': 'XML-Bare-%(version)s.tar.gz',
+        'source_urls': ['https://cpan.metacpan.org/authors/id/C/CO/CODECHILD'],
+        'checksums': ['865e198e98d904be1683ef5a53a4948f02dabdacde59fc554a082ffbcc5baefd'],
+        'patches': ['XML-Bare-0.53_icc.patch'],


should this be here?

Nice catch, of course it shouldn't be, removed in 68b2642.

Even worse, it creates problems since gcc is no longer used, resulting in a fallback to cc... 😱

…C-based toolchain

boegel · 2017-10-13T18:03:46Z

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

vanzod · 2017-10-13T18:31:44Z

Going in, thanks @boegel!

add xkeyboard-config component in X11 bundle + intltool as required b…

8d08187

…uild dep

boegel added this to the 3.4.0 milestone Aug 28, 2017

consistently include xkeyboard-config in X11 bundle + add required in…

e0e2c72

…tltool as build dep

boegel changed the title ~~add xkeyboard-config component in X11 bundle + intltool as required build dep (WIP)~~ add xkeyboard-config component in X11 bundle + intltool as required build dep (REVIEW) Sep 7, 2017

boegel added 4 commits September 7, 2017 12:44

stick with XML-Parser 2.44 with foss/2015a

2cd35dd

consistently add source URL for xkeyboard-config in X11 bundle & enha…

cbe117b

…nce sanity check

add/fix checksums in XML-Parser easyconfigs

30d943a

fix toolchain conflict for expat with gimkl/2017a

2f9e8bf

easybuilders deleted a comment from boegelbot Sep 7, 2017

boegel modified the milestones: 3.4.0, 3.4.1 Sep 8, 2017

fix sanity check for older XML-Parser

c090f3c

boegel requested a review from vanzod October 12, 2017 18:44

migueldiascosta reviewed Oct 13, 2017

View reviewed changes

don't use patch for XML-Bare that enforces use of icc when using a GC…

68b2642

…C-based toolchain

vanzod approved these changes Oct 13, 2017

View reviewed changes

vanzod merged commit 68507c6 into easybuilders:develop Oct 13, 2017

boegel deleted the X11_xkb branch October 13, 2017 18:37

boegel added a commit to boegel/easybuild-easyconfigs that referenced this pull request Oct 16, 2017

include easybuilders#5066 in release notes

b7ab9bb

Conversation

boegel commented Aug 28, 2017

Uh oh!

jhein32 commented Aug 28, 2017

Uh oh!

boegel commented Aug 28, 2017

Uh oh!

jhein32 commented Aug 28, 2017

Uh oh!

JackPerdue commented Aug 28, 2017

Uh oh!

jhein32 commented Aug 30, 2017 via email

Uh oh!

boegel commented Sep 7, 2017

Uh oh!

boegel commented Oct 12, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

boegel commented Oct 12, 2017

Uh oh!

vanzod commented Oct 12, 2017

Uh oh!

migueldiascosta Oct 13, 2017

Choose a reason for hiding this comment

Uh oh!

boegel Oct 13, 2017

Choose a reason for hiding this comment

Uh oh!

boegel commented Oct 13, 2017

Uh oh!

vanzod commented Oct 13, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

boegel commented Oct 12, 2017 •

edited

Loading