GNU bug report logs - #5475
Archives with filenames with square brackets

Previous Next

Full log

Message #41 received at 5475 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Juri Linkov <juri <at> jurta.org>
Cc: 5475 <at> debbugs.gnu.org, cyd <at> stupidchicken.com
Subject: Re: bug#5475: Archives with filenames with square brackets
Date: Sun, 31 Jan 2010 20:04:03 +0200

> From: Juri Linkov <juri <at> jurta.org>
> Cc: cyd <at> stupidchicken.com,  5475 <at> debbugs.gnu.org
> Date: Sun, 31 Jan 2010 12:56:19 +0200
> 
> >> The default value of `archive-zip-extract' can be either "unzip" or
> >> "pkunzip" (depending on the presence of the executable in `exec-path'),
> >> but the function `archive-zip-extract' compares with the value "pkzip".
> >> So the pkzip/pkunzip branch is never executed.
> >>
> >> It seems the intention of the change revno#45347 (2002-05-19) was
> >> to compare with "pkunzip" instead of "pkzip"
> >
> > Right, looks like a bug.
> >
> > Btw, I looked into 7z, and it doesn't have the same problem as unzip.
> > So we probably should only quote with unzip.
> 
> Does the following patch look right?  At least, it fixes a bug, and
> after feature freeze we could add more data structures to defcustom to
> indicate whether the file name needs to be quoted.

Looks okay to me, except that perhaps compare strings
case-insensitively instead of just with `equal'.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.