Skip to content

Conversation

@bwbarrett
Copy link
Member

@bwbarrett bwbarrett commented Jul 13, 2024

Update VERSION and NEWS files for 4.1.7 release.

bot:notacherrypick

@bwbarrett bwbarrett requested a review from jsquyres July 13, 2024 22:09
@github-actions github-actions bot added this to the v4.1.7 milestone Jul 13, 2024
@bwbarrett bwbarrett force-pushed the release/v4.1.7-prep branch from 4aa7181 to 31261f6 Compare July 13, 2024 22:49
@wenduwan
Copy link
Contributor

@jsquyres Reminder to review

@open-mpi open-mpi deleted a comment from github-actions bot Jul 25, 2024
Update VERSION and NEWS files for 4.1.7 release.

Signed-off-by: Brian Barrett <[email protected]>
@bwbarrett bwbarrett force-pushed the release/v4.1.7-prep branch from 31261f6 to d112142 Compare August 12, 2024 20:12
@bwbarrett bwbarrett merged commit 778476f into open-mpi:v4.1.x Aug 12, 2024
@bwbarrett bwbarrett deleted the release/v4.1.7-prep branch August 12, 2024 22:42
@yosefe
Copy link
Contributor

yosefe commented Jan 15, 2025

@bwbarrett Was it really the intention to bump the age of liboshmem_so_version from 30 to 31?
Seems that because of this change, liboshmem.so.40 was changed to liboshmem.so.39 (in latest build of openmpi rpm), which breaks yum upgrade.

@janjust
Copy link
Contributor

janjust commented Jan 22, 2025

@bwbarrett ^

@artemry-nv
Copy link

@jsquyres should we open a new issue for this?
liboshmem_so_version=70:3:30 -> liboshmem_so_version=70:4:31: the age change (30 -> 31) while current (70) is the same caused SONAME changes (70-30=40 -> 70-31=39) liboshmem.so.40 -> liboshmem.so.39 which as mentioned above by Yossi breaks package manager dependencies (e.g. yum update fails). Unless it's intentional, it looks like a bug.

@jsquyres
Copy link
Member

@yosefe @artemry-nv Yes, we probably need an issue for this. @bwbarrett and I have been discussing what we're going to do to fix this...

@artemry-nv
Copy link

Submitted #13051

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.

6 participants