<html>#595: unused function ParseEnvCommands in simfactory
<table style='border-spacing: 1ex 0pt; '>
<tr><td style='text-align:right'> Reporter:</td><td>Frank Löffler</td></tr>
<tr><td style='text-align:right'> Status:</td><td>resolved</td></tr>
<tr><td style='text-align:right'>Milestone:</td><td></td></tr>
<tr><td style='text-align:right'> Version:</td><td>development version</td></tr>
<tr><td style='text-align:right'> Type:</td><td>enhancement</td></tr>
<tr><td style='text-align:right'> Priority:</td><td>minor</td></tr>
<tr><td style='text-align:right'>Component:</td><td>SimFactory</td></tr>
</table>
<p>Changes (by Roland Haas):</p>
<p><table>
<tr><td>status:</td><td>resolved (was new)</td></tr>
</table></p>
<p>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.</p>
<p><strong>Keyword:</strong></p>
<p>Comment (by Roland Haas):</p>
<p>Removed in git hash <a data-is-external-link="true" href="https://bitbucket.org/simfactory/simfactory2/commits/fde327127c358b1d01e202b6ee62699cb508bca6" rel="nofollow">fde3271</a> "simsubs: remove unused and incomplete ParseEnvCommands function" of <a data-is-external-link="true" href="https://bitbucket.org/simfactory/simfactory2" rel="nofollow">simfactory2</a>.</p>
<p>The current implementation had at least 2 bugs:</p>
<ol>
<li>it was substituting the empty string for everything</li>
<li>the pattern to find <code>@ENV(...)@</code> was way to greedy and assume whitespace after which would have prevented such uses as <code>@END(SCRATCH)@/simulations</code> which would be useful eg on TACC machines</li>
</ol>
<p>Finally Cactus itself can use ENV in parameter files.</p>
<p>--<br/>
Ticket URL: <a href='https://bitbucket.org/einsteintoolkit/tickets/issues/595/unused-function-parseenvcommands-in'>https://bitbucket.org/einsteintoolkit/tickets/issues/595/unused-function-parseenvcommands-in</a></p>
</html>