Brackets should be escaped on Csh shells

[xdelaruelle]

When defining an alias containing bracket characters ([]), if module is run against csh/tcsh shells, an error is obtained.

Example modulefile:

#%Module
set osrel 80
set-alias mycommand "bsub -R \"select[type==X64LIN && osrel==$osrel]\" myapp"

Error when running module:

$ module load ./test
alias: No match.

It seems bracket characters have a special meaning on Csh shell. Thus they should be escaped when module produces shell output.

[Laurent-Besson]

On the mailing list, I proposed the following workaround:

proc charEscaped {str {charlist " \\\t{}[]|<>!;#^$&*?"'`()"}} {
return [regsub -all "([$charlist])" $str {\\1}]
}

proc charUnescaped {str {charlist " \\\t{}[]|<>!;#^$&*?"'`()"}} {
return [regsub -all "\\([$charlist])" $str {\1}]
}

where I have added "[]" in the list of char to escape. Though it solves the issue for the "set-alias", it breaks the definition of __MODULES_LMCONFLICT variable. During an "unload" command, I get some "&" (instead of ":") as separator during the different modules. This is causing severe error under tcsh

[Laurent-Besson]

Seems GitHub interface has removed some backslashes from my previous comment. To be clear, initial workaround was:

proc charEscaped {str {charlist " \\\\\t{}\[\]|<>!;#^\$&*?\"'`()"}} {
   return [regsub -all "\(\[$charlist\]\)" $str {\\\1}]
}

proc charUnescaped {str {charlist " \\\\\t{}\[\]|<>!;#^\$&*?\"'`()"}} {
   return [regsub -all "\\\\\(\[$charlist\]\)" $str {\1}]
}

[xdelaruelle]

Additional backslashes should be added to escape the brackets on the regsub command. On my way to fix this.

[xdelaruelle]

As a workaround, the puts command can be used as you control with it the shell code that is produced:

#%Module
set osrel 80
puts stdout "alias mycommand bsub\\ -R\\ \"select\\\[type==X64LIN && osrel==$osrel\\\]\"\\ myapp;"