Skip to content

Pysam - Add XZ dependency for versions > 0.12#7971

Merged
boegel merged 2 commits intoeasybuilders:developfrom
vanzod:20190328114005_new_pr_Pysam01201
May 8, 2019
Merged

Pysam - Add XZ dependency for versions > 0.12#7971
boegel merged 2 commits intoeasybuilders:developfrom
vanzod:20190328114005_new_pr_Pysam01201

Conversation

@vanzod
Copy link
Copy Markdown
Member

@vanzod vanzod commented Mar 28, 2019

From version 0.12 XZ is required as dependency. Otherwise the following error may appear:

htslib/hts_os.c(30): catastrophic error: cannot open source file "os/rand.c"
  #include "os/rand.c"

@vanzod vanzod requested a review from boegel March 28, 2019 16:40
@vanzod
Copy link
Copy Markdown
Member Author

vanzod commented Mar 28, 2019

Test report by @vanzod
SUCCESS
Build succeeded for 10 out of 10 (10 easyconfigs in this PR)
cermis - Linux debian 9.8, Intel(R) Xeon(R) CPU X5670 @ 2.93GHz, Python 2.7.13
See https://gist.github.com/059870156a8bf8192373cb44e99de9e4 for a full test report.

@vanzod vanzod added the bug fix label Mar 28, 2019
@vanzod vanzod added this to the 3.x milestone Mar 28, 2019
@vanzod
Copy link
Copy Markdown
Member Author

vanzod commented Mar 29, 2019

Test report by @vanzod
SUCCESS
Build succeeded for 10 out of 10 (10 easyconfigs in this PR)
cermis - Linux debian 9.8, Intel(R) Xeon(R) CPU X5670 @ 2.93GHz, Python 2.7.13
See https://gist.github.com/03fb4e64ec6d434ff4828bb26bfa7b5c for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot May 8, 2019
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel modified the milestones: 3.x, next release (3.9.1) May 8, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented May 8, 2019

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

@boegel
Copy link
Copy Markdown
Member

boegel commented May 8, 2019

Going in, thanks @vanzod!

@boegel boegel merged commit de9b630 into easybuilders:develop May 8, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented May 8, 2019

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

@boegel
Copy link
Copy Markdown
Member

boegel commented May 8, 2019

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

@boegel
Copy link
Copy Markdown
Member

boegel commented May 9, 2019

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants