- changed title to unused function ParseEnvCommands in simfactory
- removed comment
unused function ParseEnvCommands in simfactory
Issue #595
closed
The function ParseEnvCommands is not used in simfactory, and as far as I can tell it is not doing what I think it should do. It should either be fully implemented (but that may already be provided elsewhere), or be removed.
Keyword:
Comments (3)
-
reporter -
- changed status to resolved
- edited description
Removed in git hash fde3271 "simsubs: remove unused and incomplete ParseEnvCommands function" of simfactory2.
The current implementation had at least 2 bugs:
- it was substituting the empty string for everything
- the pattern to find
@ENV(...)@
was way to greedy and assume whitespace after which would have prevented such uses as@END(SCRATCH)@/simulations
which would be useful eg on TACC machines
Finally Cactus itself can use ENV in parameter files.
-
- changed status to closed
- Log in to comment