On 29 March 2007 16:27, Stepan Kasal wrote:
> 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 /`
Gah, I overlooked that the original version wasn't using ls_command in the
first place.
cheers,
DaveK
--
Can't think of a witty .sigline today....
