Alexander Grund
Alexander Grund
With the recent updates to the EasyConfigs this can now be merged. I did an extensive testing using the current `develop` ECs with the identified, potentially problematic ones, see https://github.com/easybuilders/easybuild-easyblocks/pull/3026#issuecomment-1986100888...
MESS-0.1.6-foss-2019b.eb requires SLATEC but the `slatec_src.tgz` is not available, at least not the right version (sha `df009d9ef9c18aae06ce68711b1ae108d3533b4f174582c3cbea0915c4fdfe01`) Given the 2019b toolchain I'd let that slide. Or does anyone have that...
I finished checking the list. In https://github.com/easybuilders/easybuild-easyblocks/pull/3026#issuecomment-1986100888 I also tested everything on our system that potentially had this issue and all installed fine. Hence I'd strongly suggest to merge both...
@Thyre Thanks for the explanation > Score-P's build system is a bit complicated. To select a compiler during `configure`, we offer several platform files, which can be selected via `--with-nocross-compiler-suite=[x]`...
> > This sounds to me like what we should use. Especially as "`ifort` compiler did support compiler instrumentation flags. Those were added to `ifx` in 2024.0." > > So...
@casparvl I updated the EasyBlock after the discussion at https://github.com/easybuilders/easybuild-easyblocks/pull/3255#discussion_r1524517112 I also noticed and fixed 2 things: - If we can't count failures we only failed if we counted ANY...
I added https://github.com/MartinsNadia/easybuild-easyblocks/pull/3 to get this done fast. It addresses the review and handles the "FindPython" (w/o version number) case.
As @bedroge retargeted this to 5.x which creates difficult conflicts due to the considerable changes during review I created #3399 combining those 15 commits into 2
> @Flamefire Everything except the test should be addressed. I see perfect thanks! Are you going to fix the test too or do you need help with that?
I rebased your branch (fixing the conflict and squashing commits) and applied the remaining suggestion (see the last 3 commits) Enabling automerge as CI should pass, if not I'll fix...