From owner-freebsd-arch Fri Jul 26 8:44: 5 2002 Delivered-To: freebsd-arch@freebsd.org Received: from mx1.FreeBSD.org (mx1.FreeBSD.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C849B37B409 for ; Fri, 26 Jul 2002 08:43:54 -0700 (PDT) Received: from smtp.noos.fr (verlaine.noos.net [212.198.2.73]) by mx1.FreeBSD.org (Postfix) with ESMTP id 7264243E75 for ; Fri, 26 Jul 2002 08:43:51 -0700 (PDT) (envelope-from root@gits.dyndns.org) Received: (qmail 37447922 invoked by uid 0); 26 Jul 2002 15:43:49 -0000 Received: from unknown (HELO gits.gits.dyndns.org) ([212.198.229.153]) (envelope-sender ) by 212.198.2.73 (qmail-ldap-1.03) with SMTP for ; 26 Jul 2002 15:43:49 -0000 Received: from gits.gits.dyndns.org (ooyafgry02etb0c8@localhost [127.0.0.1]) by gits.gits.dyndns.org (8.12.5/8.12.5) with ESMTP id g6QFhjwZ093781; Fri, 26 Jul 2002 17:43:45 +0200 (CEST) (envelope-from root@gits.dyndns.org) Received: (from root@localhost) by gits.gits.dyndns.org (8.12.5/8.12.5/Submit) id g6QFhjXd093780; Fri, 26 Jul 2002 17:43:45 +0200 (CEST) (envelope-from root) Date: Fri, 26 Jul 2002 17:43:44 +0200 From: Cyrille Lefevre To: Juli Mallett Cc: Ruslan Ermilov , Bruce Evans , Doug Barton , Mike Barcroft , arch@FreeBSD.ORG Subject: Re: Standardized make options (or no doesn't always mean no) Message-ID: <20020726154344.GB93459@gits.dyndns.org> References: <3D02AB11.F373AB4@FreeBSD.org> <20020609123557.X21758-100000@gamplex.bde.org> <20020725070145.GE56367@sunbay.com> <20020725165940.GF58642@gits.dyndns.org> <20020725170940.GA40574@sunbay.com> <20020726135923.GA89959@gits.dyndns.org> <20020726082527.A69507@FreeBSD.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20020726082527.A69507@FreeBSD.org> User-Agent: Mutt/1.3.99i Organization: ACME X-Face: V|+c;4!|B?E%BE^{E6);aI.[< List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Fri, Jul 26, 2002 at 08:25:27AM -0700, Juli Mallett wrote: > * De: Cyrille Lefevre [ Data: 2002-07-26 ] > [ Subjecte: Re: Standardized make options (or no doesn't always mean no) ] > > On Thu, Jul 25, 2002 at 08:09:40PM +0300, Ruslan Ermilov wrote: > > > On Thu, Jul 25, 2002 at 06:59:40PM +0200, Cyrille Lefevre wrote: > > > > On Thu, Jul 25, 2002 at 10:01:45AM +0300, Ruslan Ermilov wrote: > > [snip] > > > > > We'd provide the compatibility knobs that would also trigger a warning > > > > > (a .warning to be added to make(1)) that NO_FOO should now be used > > > > > instead of NOFOO (or vice versa, if we decide to go that way). > > > > > > > > .warning isn't needed, let's try w/ .BEGIN :) > > > > well, il fact, a .warning is needed for the newer knob because > > it seems you can't do something like this : > > Can't you use a double-colon rule to accomplish that? I've also tried w/o success. > Also, the diff seems reasonable, but you really do want to keep in mind > that most places where make(1) substitutes/expands variables, it will be > allocating memory. Leaks bad. re you saying that almost all occurences of : cp = Var_Subst(NULL, cp, VAR_GLOBAL, FALSE); should really be : cp2 = Var_Subst(NULL, cp, VAR_GLOBAL, FALSE); free(cp); cp = cp2; > > - /* use fprintf/exit instead of Parse_Error to terminate immediately */ > > - fprintf(stderr, "\"%s\", line %d: %s\n", fname, lineno, errmsg); > > + Parse_Error(PARSE_FATAL, "%s", errmsg); > > exit(1); > > This seems like a regression. looking at Parse_Error, there is no reason here to use fprintf instead ? Cyrille. -- Cyrille Lefevre mailto:cyrille.lefevre@laposte.net To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message