[Developers] Patch for PR 1916: ThornGuide when arrangements are links
Tom Goodale
goodale at cct.lsu.edu
Wed May 11 05:37:21 CDT 2005
Looks good. Please apply.
Could you also take a look at the other cases you've noticed ? There have
been various problems in the past with soft links, and this looks like hte
best solution to date.
Cheers,
Tom
On Fri, 29 Apr 2005, Steve White wrote:
>
> Attached find a patch that effects a fix for PR 1916
>
> patch -p0 < ThornGuide.diff
>
> Files changed
> -------------
>
> lib/sbin/ThornUtils.pm, lib/sbin/ThornList.pl
>
> (In both files, changed a FindDirectories function.)
>
> Main purpose
> ------------
>
> Links to arrangement directories are ignored by the ThornGuide build.
> This fixes that situation.
>
> Note
> ----
>
> The FindDirectories function in both files does the same thing although the
> code looked rather different. It would be best to just use the one in the
> package file. I think there may be other examples of basically identical,
> reinvented wheels in the code.
>
> The main improvement in the patch is that instead of relying on the formatted
> output of the shell command 'ls', I use the Perl commands 'opendir, readdir',
> a technique with several advantages, among which being that it is more
> robust. I see there are several other cases of the use of the 'ls' technique
> in the Cactus code--maybe these should be scrutinised.
>
>
> ------------------------------------------------------------------------
> Steve White : Programmer
> Max-Planck-Institut für Gravitationsphysik Albert-Einstein-Institut
> Am Mühlenberg 1, D-14476 Golm, Germany +49-331-567-7625
More information about the Developers
mailing list