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