From owner-cvs-gnu  Tue Jan  7 02:02:43 1997
Return-Path: <owner-cvs-gnu>
Received: (from root@localhost)
          by freefall.freebsd.org (8.8.4/8.8.4) id CAA11513
          for cvs-gnu-outgoing; Tue, 7 Jan 1997 02:02:43 -0800 (PST)
Received: from godzilla.zeta.org.au (godzilla.zeta.org.au [203.2.228.19])
          by freefall.freebsd.org (8.8.4/8.8.4) with ESMTP id CAA11497;
          Tue, 7 Jan 1997 02:02:00 -0800 (PST)
Received: (from bde@localhost) by godzilla.zeta.org.au (8.8.3/8.6.9) id UAA17963; Tue, 7 Jan 1997 20:57:10 +1100
Date: Tue, 7 Jan 1997 20:57:10 +1100
From: Bruce Evans <bde@zeta.org.au>
Message-Id: <199701070957.UAA17963@godzilla.zeta.org.au>
To: cvs-all@freefall.freebsd.org, CVS-committers@freefall.freebsd.org,
        cvs-gnu@freefall.freebsd.org, jkh@freefall.freebsd.org
Subject: Re: cvs commit:  src/gnu/usr.bin/perl/x2p util.h
Sender: owner-cvs-gnu@FreeBSD.ORG
X-Loop: FreeBSD.org
Precedence: bulk

>  Modified:    gnu/usr.bin/perl/x2p  util.h
>  Log:
>  Remove bogus redeclaration of setenv().
>  Fixes make world failure #3 for today (part of an ongoing series).

Erm, the bogusness is primarily in <stdlib.h>.  <stdlib.h> declares a
nonstandard function setenv().  x2p has its own version of setenv().
This is incompatible with the library one (it returns void instead of
int and doesn't have an `overwrite' arg and doesn't have `const' args).
This was harmless because <stdlib.h> wasn't included together with
"util.h" where perl's version is declared.  x2p doesn't actually use
setenv() amyway.  An include of <stdlib.h> is now generated by yacc.
This is reasonable because the generated code now calls realloc().
The yacc changes came from Lite2.  x2p's setenv() seems to be bloat
from ../perl.  In ../perl, the name collision is handled by renaming
perl's setenv() to my_setenv().

Bruce