Skip to content

{vis} [intel-2015a] ParaView 4.3.1 (REVIEW) #1627

Merged
boegel merged 14 commits intoeasybuilders:developfrom
hajgato:paraview431
Oct 15, 2015
Merged

{vis} [intel-2015a] ParaView 4.3.1 (REVIEW) #1627
boegel merged 14 commits intoeasybuilders:developfrom
hajgato:paraview431

Conversation

@hajgato
Copy link
Copy Markdown
Collaborator

@hajgato hajgato commented May 21, 2015

Needs #1618

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3078/
Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3078/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.

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3205/
Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3205/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.

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3739/
Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3739/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.

@hpcugentbot
Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3741/
Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3741/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.

@hajgato hajgato changed the title {vis} [intel-2015a] ParaView 4.3.1 (WIP) {vis} [intel-2015a] ParaView 4.3.1 (REVIEW) Aug 26, 2015
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.

normally, we would add python etc, but this is just too much 😉

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.

well, if Python is a dep (and other deps also use Python), it's good practice to include -Python-<version> in the versionsuffix

@wpoely86
Copy link
Copy Markdown
Member

lgtm

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.

pls bump to 2.7.10? (as in #1618 now)

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.

yeah but, if you do python, why not the others?
-Python-2.7.10-GLib-2.44.0-libX11-1.6.3
It's ugly but either we stick with the rule or we don't

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.

because Python is a very common dep, it helps users identifying which modules are compatible with each other

the others are far less common, so listing everything in the versionsuffix yields no added benefit, it just makes things uglier

it's a pragmatic choice

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

problem is, if you really mean it to be consistent with all, it is needed also to add build modes/flags, opt options etc; we merely end up with a hash function of some build configuration. See, #1689 for another variation of this issue/need. The faster we agree that this is simply a hash of the build setup, the sooner we'll come to agreement about what it should look like.

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4611/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.

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/4614/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.

@hajgato
Copy link
Copy Markdown
Collaborator Author

hajgato commented Oct 2, 2015

@boegel made it python-2.7.10, but I find a bit ridicolous, like Qt-4.8.6-intel-2015a-GLib-2.44.1-Python-2.7.10.eb and GLib-2.44.1-intel-2015a-Python-2.7.10.eb because of python builddep.....

@hpcugentbot
Copy link
Copy Markdown

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

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

@wpoely86
Copy link
Copy Markdown
Member

wpoely86 commented Oct 5, 2015

== 2015-10-05 19:31:02,797 main.filetools WARNING IOError occurred while trying to download ftp://ftp.freedesktop.org/pub/mesa/glu/glu-9.0.0.tar.bz2 to /user/data/gent/vsc400/vsc40023/EasyBuild/sources/l/libGLU/glu-9.0.0.tar.bz2: <urlopen error ftp error: [Errno 113] No route to host>

Weird, it works here.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 5, 2015

@wpoely86: it's just because ftp:// is blocked on the HPC-UGent systems, will do a rerun tomorrow

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.

space after ,

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 6, 2015

Test report by @boegel
FAILED
Build succeeded for 5 out of 6 (6 easyconfigs in this PR)
Linux centos linux 7.1.1503, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/da0404c96bbd69eb43cc for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 6, 2015

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in this PR)
Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/b717b8ed46b3ffd1175a 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/4661/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/4664/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.4.0 milestone Oct 13, 2015
@hajgato
Copy link
Copy Markdown
Collaborator Author

hajgato commented Oct 13, 2015

@boegel: there are two ways to solve the problems:

  1. We disable the test (uncomment configopts += '-DBUILD_TESTING=OFF ')
  2. we decrease the maxparallel option

Problems are usually cannot connect, do not find file, etc. In the test phase many files are downloaded, and sometimes their server is too busy. I am in favor of 1, because compiling already takes forever.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 13, 2015

@hajgato: pre-downloading the files needed for the test (sequentially rather than in parallel) is not an option?

Can you update the PR to disable testing (option 1), and include a clear comment why you disable testing?

@hajgato
Copy link
Copy Markdown
Collaborator Author

hajgato commented Oct 13, 2015

@boegel the ; was delibeartely used instead of && in prebuildopts. So we have 2 trials to download ExternalData from the internet.

If this work, then I will implememt the same for Paraview 4.4.0 and I will rename Paraview to ParaView

@hpcugentbot
Copy link
Copy Markdown

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

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.

please add a comment here why you include is, and mention why you use the ; (i.e., to allow this command to fail, and trigger a 2nd try via make)

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 14, 2015

Test report by @boegel
FAILED
Build succeeded for 4 out of 6 (6 easyconfigs in this PR)
Linux centos linux 7.1.1503, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/c9d75834f7776de0f052 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/4736/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 Oct 15, 2015

@boegel %% fixed, extra comment added for ;

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 15, 2015

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in this PR)
Linux centos linux 7.1.1503, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/f30fee6a7e6b884a80ee for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 15, 2015

Test report by @boegel
SUCCESS
Build succeeded for 6 out of 6 (6 easyconfigs in this PR)
Linux centos linux 7.1.1503, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/4d37158008b619ac25ce for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 15, 2015

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 15, 2015

Going in, thanks @hajgato!

boegel added a commit that referenced this pull request Oct 15, 2015
{vis} [intel-2015a] ParaView 4.3.1 (REVIEW)
@boegel boegel merged commit 05e64fb into easybuilders:develop Oct 15, 2015
@hajgato hajgato deleted the paraview431 branch June 8, 2017 13:20
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.

5 participants