This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patch to remove putenv definition from gcc.c
- To: ghazi at caip dot rutgers dot edu (Kaveh R. Ghazi)
- Subject: Re: Patch to remove putenv definition from gcc.c
- From: "John David Anglin" <dave at hiauly1 dot hia dot nrc dot ca>
- Date: Wed, 28 Mar 2001 14:39:51 -0500 (EST)
- Cc: gcc-patches at gcc dot gnu dot org
> Let me know what you find.
Just sent.
> Assuming vax-vms *doesn't* have putenv, I'm not sure leaving a whole
> putenv implementation in gcc.c only to #ifdef out the entire function
> body for VMS is the right way to elide this call.
>
> A much cleaner solution would be to do "#define putenv(x)" in
> vax/xm-vms.h.
Yes, that's a better approach than what I just suggested. As far
as I can tell, this is a VAX C version issue rather than a difference
between vax and alpha implementations. Thus, I think both the vax
and alpha xm-vms.h files need some modification. Maybe
/* Early versions of VAX C for VMS do not have putenv. Comment out
the following define if your system doesn't have putenv. */
#define HAVE_PUTENV
#ifndef HAVE_PUTENV
#define putenv(x)
#endif
> So if vax-vms has putenv, I'll install my patch as-is, otherwise I'd
> like to simply add the above macro to the original patch.
>
> Sound okay?
Yes with the above revision.
Dave
--
J. David Anglin dave.anglin@nrc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6605)