On Wed, Dec 05, 2007 at 05:45:46PM +0000, Colin Watson wrote:
> > In the event there IS an empty string in handed to the loop on line 46,
> > nothing will happen unless the argument ("$PROGRAM") happens to exist as
> > a file in / with executable permissions (assuming you remove lines
> > 47-49, otherwise it searches the current directory).
> 
> I believe that Clint is incorrect that this was the intent; it certainly
> wasn't my intent, as that does not correspond to the shell's command
> search semantics. It's just an unremarkable accident of the obvious
> implementation. ELEMENT is not supposed to be empty in that case branch.

I'm afraid I'm either misunderstanding the original message or Colin's
reply, though I'm not sure it matters for practical disposition of this
bug.

> So you need "*[!:]:" there.

> Please fix this by changing the pattern to "*[!:]:", or back to the
> perfectly good and arguably clearer (if longer) code that was there
> before.

Aye.



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to