On Oct 2, 2007, at 1:20 AM, Paul Eggert wrote:
Ralf Wildenhues [EMAIL PROTECTED] writes:
+ xsystem (rm -rf '$cache')
This won't work well if $cache contains an apostrophe.
I realize this sort of problem is not new to the patch, but while
we're making changes in this area, why not fix
Ralf Wildenhues [EMAIL PROTECTED] writes:
+ xsystem (rm -rf '$cache')
This won't work well if $cache contains an apostrophe.
I realize this sort of problem is not new to the patch, but while
we're making changes in this area, why not fix it? Either check for
and reject cache names
Hello Benoit,
Ooops. It seems my memory was messed up, thinking it was your turn
of work with this patch and not mine ... apologies for the long delay,
and thanks for the revised patch.
* Benoit Sigoure wrote on Mon, Jun 25, 2007 at 12:05:31AM CEST:
Alright so here is the new (hopefully
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
According to Ralf Wildenhues on 9/30/2007 10:21 AM:
+{
+ unlink $f or error error: Cannot remove `$f': $!
+unless not -f $f;
FWIW, this kind of code leaves me wondering about perl operator
precedence. Is `unless' an operator
Quoting Ralf Wildenhues [EMAIL PROTECTED]:
Hello Benoit,
* Benoit Sigoure wrote on Sun, Jun 17, 2007 at 01:16:35AM CEST:
I've reviewed the patches I proposed back in March, completed them with
tests, ChangeLog and NEWS entries. If they are accepted, I'll write the
documentation.
Please