Skip to content

GC3Pie/2.5.0 - Add missing dependencies#7066

Merged
boegel merged 3 commits intoeasybuilders:developfrom
vanzod:20181022173809_new_pr_GC3Pie250
Oct 28, 2018
Merged

GC3Pie/2.5.0 - Add missing dependencies#7066
boegel merged 3 commits intoeasybuilders:developfrom
vanzod:20181022173809_new_pr_GC3Pie250

Conversation

@vanzod
Copy link
Copy Markdown
Member

@vanzod vanzod commented Oct 22, 2018

(created using eb --new-pr)

@vanzod
Copy link
Copy Markdown
Member Author

vanzod commented Oct 22, 2018

Test report by @vanzod
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
cermis - Linux debian 9.5, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.7.13
See https://gist.github.com/5530d63931ad4ac194d07d8a867ce66f for a full test report.

@vanzod vanzod changed the title Add missing dependencies GC3Pie/2.5.0 - Add missing dependencies Oct 22, 2018
@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 22, 2018

Hmm, how come we missed this with exts_download_dep_fail = True set?

@boegel boegel added this to the 3.8.0 milestone Oct 22, 2018
@boegel boegel added the bug fix label Oct 22, 2018
@vanzod
Copy link
Copy Markdown
Member Author

vanzod commented Oct 23, 2018

@boegel That's because it is built with the system Python. If you have those packages already installed they are not downloaded at build time. This is the case of my dev box and I noticed the issue only on the production cluster where those packages are not installed from the OS repo.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 23, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
gligar02.gligar.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/02417b6e5310a4c11a650d5f5ca0662e for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 25, 2018

@vanzod GC3Pie-2.5.0.eb:142:6: E201 whitespace after '(' ;)

@easybuilders easybuilders deleted a comment from boegelbot Oct 28, 2018
@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 28, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node3101.skitty.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/b3047ecb293df16760e5f6abd2851b4d for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 28, 2018

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 28, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2007.delcatty.os - Linux centos linux 7.5.1804, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/9278717e4743893c4b6077a96fd6cf5b for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 28, 2018

@vanzod If @riccardomurri's patch from easybuilders/easybuild-framework#2632 checks out, maybe we should apply it in this easyconfig as well (in a follow-up PR)?

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 28, 2018

Going in, thanks @vanzod!

@boegel boegel merged commit f55e13e into easybuilders:develop Oct 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants