BUG: EscapePerlString too eager

[schumar]

EscapePerlString in utility.c,
introduced in 14c9aaf and expanded in 5a6e32a, escapes

• single-quotes (correct)
• backslashes (NOT correct)
• semi-colons (NOT correct)

If a module-files has

setenv foo bar\\zak\;gar

it will correctly set the env-var foo to bar\zak;gar in e.g. bash, but will result in bar\\zak\;gar in Perl:

$ modulecmd perl load $module
[...] $ENV{'foo'} = 'bar\\zak\;gar';

(The current branch (since 8e050c4) only escapes single-quotes:

$ modulecmd perl load $module
[...]
$ENV{'foo'} = 'bar\zak;gar';

i.e. does not have this bug.)

[xdelaruelle]

Thanks for the contribution. Despite the message on this github issue, I have accepted this code proposal. I just changed it a bit to adapt the non-regression test suite.

For the record: issue is marked "closed" rather "merged" as I messed up with the remote branch. But everything is correctly merged.