-
Notifications
You must be signed in to change notification settings - Fork 772
add missing OS dependencies in Java 1.8 easyconfig used on POWER systems #9454
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
add missing OS dependencies in Java 1.8 easyconfig used on POWER systems #9454
Conversation
|
@Flamefire (or @branfosj) Are you up for submitting a test report for this on a POWER system? |
|
Test report by @branfosj |
|
Currently running (again) Just wanted to highlight the option |
|
Test report by @Flamefire |
|
@Flamefire Adding @branfosj Any objections to the changes made here? |
Would this be for this particular Java easyconfig or everything? On our site we install software with GUI components on our windowing-capable login nodes. It's not unfathomable that some of those UIs may be in Java, and it's definitely an aspect of "HPC" that EasyBuild tends to cover. |
|
I'd prefer not adding the |
|
Sure. What would the right osdependencies be in this case? The build suggests |
|
Test report by @Flamefire |
This sounds like at least: osdependencies = ['libXtst-devel', 'libXt-devel', 'libXrender-devel']Ideally also the equivalent package name for non-RedHat based Linux distros are added as alternatives by promoting the single string value to a tuple with alternative package names, but I'm not sure that's worth the effort here (is anyone brave enough to use a non RHEL-based distro on POWER)? |
|
They are: |
So that would be |
Exactly. |
bb9bd39 to
02dd194
Compare
|
Ok done. I added the other debian packages too and squashed the commits |
02dd194 to
abda90d
Compare
|
Test report by @Flamefire |
|
Going in, thanks @Flamefire! |
(created using
eb --new-pr)Fixes #9452