Skip to content

{vis}[intel-2016a] FLTK 1.3.3 with libX11 1.6.3 (REVIEW)#2645

Merged
boegel merged 3 commits intoeasybuilders:developfrom
hajgato:FLTK-X
Mar 8, 2016
Merged

{vis}[intel-2016a] FLTK 1.3.3 with libX11 1.6.3 (REVIEW)#2645
boegel merged 3 commits intoeasybuilders:developfrom
hajgato:FLTK-X

Conversation

@hajgato
Copy link
Copy Markdown
Collaborator

@hajgato hajgato commented Mar 7, 2016

No description provided.

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/6836/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 boegel added this to the v2.7.0 milestone Mar 7, 2016

libx11 = 'libX11'
libxver = '1.6.3'
versionsuffix = '-%s-%s' % (libx11, libxver)
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.

I'm not sure if this is necessary?

It's OK to include libX11 as a dep and not use a versionsuffix imho...

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@boegel dunno. It is a drastic change, but I can leave it out.

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.

I would leave it out...

@hpcugentbot
Copy link
Copy Markdown

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

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/6870/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
Copy link
Copy Markdown
Collaborator Author

hajgato commented Mar 8, 2016

@boegel: Tcl/Tk dep removed

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 8, 2016

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/f3962d4dc01fdc6c688c for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 8, 2016

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 8, 2016

Going in, thanks @hajgato!

boegel added a commit that referenced this pull request Mar 8, 2016
{vis}[intel-2016a] FLTK 1.3.3 with libX11 1.6.3 (REVIEW)
@boegel boegel merged commit 2d4b2fa into easybuilders:develop Mar 8, 2016
@hajgato hajgato deleted the FLTK-X branch June 8, 2017 13:15
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.

3 participants