[bugfix] Fix bug when patchlevel in environment modules version is not an integer

[jonathanfrawley]

A system I am working on uses version numbers such as "3.2.1+26-ad12ae47" for the module environment. This breaks reframe, but since reframe only looks at the major and minor numbers, it is not required to parse out the bugfix version at all.

[jenkins-cscs]

Can I test this patch?

[codecov-io]

Codecov Report

Merging #1540 into master will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #1540   +/-   ##
=======================================
  Coverage   87.90%   87.90%           
=======================================
  Files          43       43           
  Lines        6848     6848           
=======================================
  Hits         6020     6020           
  Misses        828      828           

Impacted Files	Coverage Δ
reframe/core/modules.py	60.36% <0.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a567e70...170d505. Read the comment docs.

[vkarak]

Hi @jonathanfrawley, thanks for the PR and the fix. We will also need a unit test to reproduce the bug. I can provide one.

[vkarak]

Hi @jonathanfrawley, thanks for the PR and the fix. We will also need a unit test to reproduce the bug. I can provide one.

Ignore my comment about the unit tests. I just realised it was in the way we initialize the modules backend, so it should be fine if you just address my other comment.

[vkarak]

maxsplit=2 won't work, so lgtm as it is now.

[vkarak]

ok to test

[victorusu]

lgtm

[vkarak]

@jenkins-cscs retry tsa