Skip to content

add patch for UCX v1.11.2 to fix missing include statement for string.h#14317

Merged
Micket merged 4 commits intoeasybuilders:developfrom
sassy-crick:20211110134850_new_pr_UCX1112
Nov 12, 2021
Merged

add patch for UCX v1.11.2 to fix missing include statement for string.h#14317
Micket merged 4 commits intoeasybuilders:developfrom
sassy-crick:20211110134850_new_pr_UCX1112

Conversation

@sassy-crick
Copy link
Copy Markdown
Collaborator

(created using eb --new-pr)

Comment thread easybuild/easyconfigs/u/UCX/UCX-1.11.2-missing_header.patch
Comment thread easybuild/easyconfigs/u/UCX/UCX-1.11.2-GCCcore-11.2.0.eb Outdated
@Micket Micket added the bug fix label Nov 10, 2021
@Micket Micket added this to the next release (4.5.1?) milestone Nov 10, 2021
Comment thread easybuild/easyconfigs/u/UCX/UCX-1.11.2-missing_header.patch Outdated
Comment thread easybuild/easyconfigs/u/UCX/UCX-1.11.2-missing_header.patch Outdated
@sassy-crick sassy-crick changed the title Added patch for missing headers for Ubuntu-16.04.7 LTS {lib}[GCCcore/11.2.0] UCX-1.11.2 Nov 10, 2021
@SebastianAchilles
Copy link
Copy Markdown
Member

Test report by @SebastianAchilles
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jsfc01.int.jusuf.sebastian.cluster - Linux rocky linux 8.4, x86_64, AMD EPYC 7742 64-Core Processor (zen2), 1 x NVIDIA GRID V100-4C, 460.73.01, Python 3.6.8
See https://gist.github.com/752880d6a3dea2a97a1238acad8407ef for a full test report.

Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

LGTM

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Nov 12, 2021

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

@Micket: Request for testing this PR well received on login1

PR test command 'EB_PR=14317 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_14317 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 967116273 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

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cnx1 - Linux rocky linux 8.4, x86_64, Intel(R) Xeon(R) CPU E5-2690 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/82f630c8be34ec69d5349cc382e7a0a9 for a full test report.

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Nov 12, 2021

Test report by @Micket
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
alvis-c1 - Linux centos linux 7.9.2009, x86_64, Intel Xeon Processor (Skylake), Python 3.6.8
See https://gist.github.com/2b9ae658f559eb9c242e352c4c1b860c for a full test report.

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Nov 12, 2021

Going in, thanks @sassy-crick!

@Micket Micket merged commit c0bc506 into easybuilders:develop Nov 12, 2021
@sassy-crick sassy-crick deleted the 20211110134850_new_pr_UCX1112 branch November 12, 2021 15:24
@boegel boegel changed the title {lib}[GCCcore/11.2.0] UCX-1.11.2 add patch for UCX v1.11.2 to fix missing include statement for string.h Nov 12, 2021
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.

5 participants