Message ID | 83txofi24i.fsf@gnu.org |
---|---|
State | New |
Headers | show |
On 3/13/13, Eli Zaretskii <eliz@gnu.org> wrote: > > #ifdef __MSVCRT__ > extern _CRTIMP char *** __cdecl __MINGW_NOTHROW __p__environ(void); > extern _CRTIMP wchar_t *** __cdecl __MINGW_NOTHROW > __p__wenviron(void); > # define _environ (*__p__environ()) > # define _wenviron (*__p__wenviron()) > #else /* ! __MSVCRT__ */ > #endif /* ! __MSVCRT__ */ > > #define environ _environ Cool. > and setenv.c does this: > > #ifndef HAVE_ENVIRON_DECL > extern char **environ; > #endif > > Solution: Add a guard: This is OK with a ChangeLog entry. Thanks. Ian
--- libiberty/setenv.c~ 2011-02-03 09:23:59.000000000 +0200 +++ libiberty/setenv.c 2013-03-13 13:22:49.085187200 +0200 @@ -63,8 +63,10 @@ #define __environ environ #ifndef HAVE_ENVIRON_DECL +#ifndef environ extern char **environ; #endif +#endif OK to commit (with a suitable ChangeLog entry)?