On Sat, Oct 23, 2010 at 8:35 PM, Olaf van der Spek <[email protected]> wrote: > On Wed, Oct 20, 2010 at 1:18 PM, Didar Hossain <[email protected]> > wrote: >>> Patch attached. Please, test it and let me know. >>> >>> Currently there is one redundant call `chdir' - but, I didn't want to >>> mess with the script >>> much. >> >> Wrong patch got attached in the previous mail. This is the correct one. > > Hmm, it's way easier then I thought, thanks. > I've reduced the patch to: > - if (symlink($available{$do}, $target)) { > + if (symlink("../conf-available/" . > basename($available{$do}), $target)) {
I think it is also best to move the chdir() call out of the "foreach" loop. Please, put the chdir() call before the foreach statement - Didar -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
