Skip to content

add missing UnZip dependency for Python 3.8.2#11458

Merged
boegel merged 4 commits intoeasybuilders:developfrom
terjekv:20201011114344_new_pr_Python382
Oct 12, 2020
Merged

add missing UnZip dependency for Python 3.8.2#11458
boegel merged 4 commits intoeasybuilders:developfrom
terjekv:20201011114344_new_pr_Python382

Conversation

@terjekv
Copy link
Copy Markdown
Collaborator

@terjekv terjekv commented Oct 11, 2020 •
edited by boegel
Loading

(created using eb --new-pr)

requires #11457

@terjekv
Copy link
Copy Markdown
Collaborator Author

terjekv commented Oct 11, 2020

Also fixes the issue here: #11417 (comment)

@terjekv
Copy link
Copy Markdown
Collaborator Author

terjekv commented Oct 12, 2020

Test report by @terjekv
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
ninhursaga.uio.no - Linux RHEL 8.2, x86_64, Intel(R) Xeon(R) Silver 4216 CPU @ 2.10GHz (cascadelake), Python 3.6.8
See https://gist.github.com/a912e35ddfb2210b0fe5da5557e57ef4 for a full test report.

boegel
boegel reviewed Oct 12, 2020
View reviewed changes
Comment thread easybuild/easyconfigs/p/Python/Python-3.8.2-GCCcore-9.3.0.eb Outdated
@boegel boegel added this to the next release (4.3.1) milestone Oct 12, 2020
@boegel boegel added the bug fix label Oct 12, 2020
@easybuilders easybuilders deleted a comment from boegelbot Oct 12, 2020
@boegel boegel changed the title Depend on UnZip. Requires http://github.com/easybuilders/easybuild-easyconfigs/pull/11457 add missing UnZip dependency for Python 3.8.2 Oct 12, 2020
@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 12, 2020

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Oct 12, 2020

@boegel: Request for testing this PR well received on generoso

PR test command 'EB_PR=11458 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_11458 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 8056

Test results coming soon (I hope)...

Details

- notification for comment with ID 707201370 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Oct 12, 2020

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
generoso-x-2 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/b428a80905b70afcb3689b74b34c428a for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 12, 2020

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 12, 2020

Going in, thanks @terjekv!

@boegel boegel merged commit 7d0da55 into easybuilders:develop Oct 12, 2020
This was referenced Oct 12, 2020
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.3.1

Development

Successfully merging this pull request may close these issues.

3 participants

@terjekv @boegel @boegelbot