Re: mdate-sh borks on uid/gid containing a space

[Stepan Kasal]

Hello again,

On Thu, Mar 29, 2007 at 04:12:11PM +0100, Dave Korn wrote:
> On 29 March 2007 15:38, Stepan Kasal wrote:
> > On Thu, Mar 29, 2007 at 12:40:12AM +0100, Dave Korn wrote:
> >> On 29 March 2007 00:26, Ralf Wildenhues wrote:
> >>> <http://thread.gmane.org/gmane.comp.sysutils.automake.patches/2730/focus=2740>
> >>> Eric's patch seems to indicate that line 92 needs changed, too.
[...]
> > So it is safer to use -n with ls /, too.
> 
>   Umm, yes, that's true, but not relevant; what I was pointing out is that the
> -n gets added to the $ls_command variable, so it would already be included in
> the command invoked on line 92, and if you take a second look at the patch,
> you'll notice that the change to line 92 isn't about adding -n, it's about
> taking away -l and -d.

I took a second look, and it did not help:

@@ -89,7 +93,7 @@
 # words should be skipped to get the date.

 # On HPUX /bin/sh, "set" interprets "-rw-r--r--" as options, so the "x" below.
-set x`ls -l -d /`
+set x`$ls_command /`

 # Find which argument is the month.
 month=

Cheers,
        Stepan