[ET Trac] [Einstein Toolkit] #605: Simfactory does not find the right 'path' for symlinked cactus directories
Einstein Toolkit
trac-noreply at einsteintoolkit.org
Fri Oct 7 11:34:31 CDT 2011
#605: Simfactory does not find the right 'path' for symlinked cactus directories
-------------------------+--------------------------------------------------
Reporter: knarf | Owner: eschnett
Type: defect | Status: review
Priority: minor | Milestone: ET_2011_10
Component: SimFactory | Version: development version
Resolution: | Keywords:
-------------------------+--------------------------------------------------
Comment (by barry.wardell):
Replying to [comment:4 eschnett]:
> Simfactory doesn't really use "chdir". There are a few chdir commands
surrounding calls to external programs (e.g. before calling the
runscript), and these should probably be replaced by explicit "cd"
commands in the executed commands.
>
> That is, constructs such as
> {{{
> orig = os.getcwd()
> chdir (somewhere)
> system "do-something"
> chdir (orig)
> }}}
> should be replaced by
> {{{
> system "cd somewhere && do-somethig"
> }}}
> which are shorter, and where there is no chdir involved.
In these cases, would it not be better to just use the cwd option to
popen()?
--
Ticket URL: <https://trac.einsteintoolkit.org/ticket/605#comment:5>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit
More information about the Trac
mailing list