From owner-freebsd-bugs Mon May 11 03:58:31 1998
Return-Path: <owner-freebsd-bugs@FreeBSD.ORG>
Received: (from majordom@localhost)
by hub.freebsd.org (8.8.8/8.8.8) id DAA19466
for freebsd-bugs-outgoing; Mon, 11 May 1998 03:58:31 -0700 (PDT)
(envelope-from owner-freebsd-bugs@FreeBSD.ORG)
Received: from mail.cs.tu-berlin.de (root@mail.cs.tu-berlin.de [130.149.17.13])
by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id DAA19405;
Mon, 11 May 1998 03:57:58 -0700 (PDT)
(envelope-from wosch@cs.tu-berlin.de)
Received: from caramba.cs.tu-berlin.de (wosch@caramba.cs.tu-berlin.de [130.149.17.12])
by mail.cs.tu-berlin.de (8.8.8/8.8.8) with ESMTP id MAA26209;
Mon, 11 May 1998 12:07:23 +0200 (MET DST)
Received: (from wosch@localhost)
by caramba.cs.tu-berlin.de (8.8.8/8.8.8) id MAA09020;
Mon, 11 May 1998 12:07:20 +0200 (MET DST)
Message-ID: <19980511120719.A7813@caramba.cs.tu-berlin.de>
Date: Mon, 11 May 1998 12:07:19 +0200
From: Wolfram Schneider <wosch@cs.tu-berlin.de>
To: Poul-Henning Kamp <phk@critter.freebsd.dk>
Cc: wosch@FreeBSD.ORG, freebsd-bugs@FreeBSD.ORG,
freebsd-gnats-submit@FreeBSD.ORG
Subject: Re: bin/6550
References: <19980510223247.09270@panke.de> <14163.894875793@critter.freebsd.dk>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
In-Reply-To: <14163.894875793@critter.freebsd.dk>; from Poul-Henning Kamp on Mon, May 11, 1998 at 10:36:33AM +0200
Sender: owner-freebsd-bugs@FreeBSD.ORG
Precedence: bulk
X-Loop: FreeBSD.org
On 1998-05-11 10:36:33 +0200, Poul-Henning Kamp wrote:
> In message <19980510223247.09270@panke.de>, Wolfram Schneider writes:
> >On 1998-05-10 12:57:25 -0700, Poul-Henning Kamp wrote:
> >> Synopsis: make(1): shell meta-character optimization incomplete
> >>
> >> State-Changed-From-To: open-closed
> >
> >*grumble*
> >
> >Our make is broken, period! Why did you closed the PR?
> >Please fix the bugs or reopen the PR again!
>
> Our make is not broken (at least in this particular respect.)
It is broken.
> There is not really any reason (apart from speed) for implementing
> the shell commands, consequently it cannot be said to be broken
> because it doesn't.
You did not understand the problem ;-(
The current algorithm is
if (no_meta_characters(command))
exec(command)
else
/bin/sh -c "command"
where it should be
if (no_meta_characters(command) && no_shell_builtin(command))
exec(command)
else
/bin/sh -c "command"
> If you want it to support more builtins, something which I am not
> as such against, while on the other hand I don't see any reason to
> do it, the very least you can do is to submit a patch.
I got now a patch from Theo de Raadt. I will check if the
patch works.
--
Wolfram Schneider <wosch@freebsd.org> http://www.freebsd.org/~wosch/
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-bugs" in the body of the message