I made sure that Lmod 5.2rc4 (or any other rc release) is not accepted, because that version of Lmod still contains bugs. From the unit tests:

EasyBuildError: 'EasyBuild crashed with an error (at easybuild/tools/modules.py:663 in update): An error occured when running \'spider -o moduleT /Users/kehoste/work/easybuild-framework/test/framework/modules\': \nLmod has detected the following error: /Users/kehoste/work/easybuild-framework/test/framework/modules/GCC/.modulerc: (GCC/.modulerc): invalid command name "module-version"\n'

ok

We're currently NOT using eb to reflect to spider that the cache needs updating as of now, but might be a nice gimmick at some point in time.

Looks good. I will review this on our production system but need to bump to 5.2 release that came out today first.

@pforai: I bumped Lmod to v5.2 on our Jenkins test server, and the framework unit tests now fail on top of Lmod because this PR isn't merged in yet (unless I set $LMOD_IGNORE_CACHE, then all is well).

So, this PR is getting a bit critical.

Please let me know when you've managed to test things on your end. Any ETA for that?

Still doesn't work if I don't set LMOD_IGNORE_CACHE=1. Investigating....

Edit:
Was my bad, works fine now without me setting LMOD_IGNORE_CACHE=1!

@pforai: So, good to merge in?

@pfo confirmed on IRC this is good to go, so merging it in!