Re: [PATCH] Revert "compat/unsetenv.c: Fix a sparse warning"
Benoit Sigoure <tsunanet@gmail.com> writes:
Show 10 quoted lines
> This reverts commit ec535cc27e6c4f5e0b1d157e04f5511f166ecd9d.
>
> POSIX explicitly states "the [environ] variable, which
> must be declared by the user if it is to be used directly".
> Not declaring it causes compilation to fail on OS X.
>
> Instead don't declare the variable on MinGW, as it causes
> a spurious warning there.
>
> Signed-off-by: Benoit Sigoure <tsunanet@gmail.com>
Show 20 quoted lines
> ---
>
> Resending as I forgot to Sign-off the previous patch.
>
> compat/unsetenv.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/compat/unsetenv.c b/compat/unsetenv.c
> index 4ea1856..bf5fd70 100644
> --- a/compat/unsetenv.c
> +++ b/compat/unsetenv.c
> @@ -2,6 +2,9 @@
>
> void gitunsetenv (const char *name)
> {
> +#if !defined(__MINGW32__)
> + extern char **environ;
> +#endif
> int src, dst;
> size_t nmln;