-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pin jemalloc to v5.2.0 for install package #37405
Pin jemalloc to v5.2.0 for install package #37405
Conversation
It is not included in the package dependencies. This also sorts the dependencies so diff shows more meaningful information when comparing the platforms.
v5.2.1 has an issue for some tests, v5.3.0 doesn't want to work on rhel9
Can we point this at |
Linux package build 1860 is resubmitted against |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a small typo that will probably affect macOS packaging
Co-authored-by: thomashampson <thomas.hampson@stfc.ac.uk>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy for this go in to resolve issues on rhel9, and work on a rocky8 fix separately
This makes the
mantid
package installer match the version of jemalloc that the developers are using.Refs #36220 .
Further detail of work
The main change is to pin the version of jemalloc for the package to v5.2.0. It is unclear what is wrong with the newer versions, but comparing the recipe from v5.2.0 and recipe from v5.3.0 one will immediately discover that v5.3.0 reworked the packages so there is now a
jemalloc
packagelibjemalloc
package where v5.2.0 only hasjemalloc
package. More investigation will need to be done to figure out how to move forward from v5.2.0.The oddity is that both v5.2.0 and v5.3.0 work on rhel7, but only v5.2.0 works on rocky8 and rhel9.
There is some extra cleanup of the developer dependencies so they can more easily be compared using
diff
(sorting), removing duplicate dependencies (conda-wrappers
twice in windows), removing unused osx dependency (jemalloc
is not runtime for mac).Bonus information: The abi compatibility chart for jemalloc doesn't go all the way to v5.3.0.
To test:
Install the built linux package on newer rhel and see that it can start and show the instrument view with opengl.
This does not require release notes because it is fixing a bug within the release cycle discovered when moving from rhel7.
Reviewer
Please comment on the points listed below (full description).
Your comments will be used as part of the gatekeeper process, so please comment clearly on what you have checked during your review. If changes are made to the PR during the review process then your final comment will be the most important for gatekeepers. In this comment you should make it clear why any earlier review is still valid, or confirm that all requested changes have been addressed.
Code Review
Functional Tests
Does everything look good? Mark the review as Approve. A member of
@mantidproject/gatekeepers
will take care of it.Gatekeeper
If you need to request changes to a PR then please add a comment and set the review status to "Request changes". This will stop the PR from showing up in the list for other gatekeepers.