mirror of
https://github.com/envmodules/modules.git
synced 2026-06-18 00:06:53 +08:00
Until now some module sub-commands did not finish their run with an ok statement when everything goes well. It was the case for avail, aliases, list, apropos/search/keyword, save*, init* and help (without arg). However when some errors occurred on these commands, they were returning an error statement. To apply same behavior to all commands whatever the result is (error occurred or not) a call to renderSettings is made after all commands which will produce a true or false statement sent to stdout. As a consequence the return value of all module sub-command call can be checked against a true/false value. Even if modulecmd.tcl should now return a true/false statement, keep checks on the intermediate variable required on python and cmake shells to guaranty a false boolean return in case very bad stuff happened within modulecmd.tcl not producing the intermediate variable statement. Adapt all non-reg tests to get correct return code (OK or ERR). Note that when apropos/search/keyword hit a modulefile error when interpreting, it will return a 'false' statement even if result output seem correct (as these kind of error are not hidden to better read obtained result).