chicken-hackers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Chicken-hackers] [PATCH] Fix unsetenv on Windows


From: Peter Bex
Subject: Re: [Chicken-hackers] [PATCH] Fix unsetenv on Windows
Date: Sat, 9 Nov 2013 16:49:44 +0100
User-agent: Mutt/1.4.2.3i

On Sat, Nov 09, 2013 at 03:44:03PM +0000, Mario Domenech Goulart wrote:
> On Sat, 9 Nov 2013 16:30:35 +0100 Peter Bex <address@hidden> wrote:
> > On Sat, Nov 09, 2013 at 03:26:21PM +0000, Mario Domenech Goulart wrote:
> >> 
> >> Environment variables were not really being unset on Windows.
> >
> > Why does the patch use setenv() with an empty string, instead of
> > unsetenv()?  Doesn't Windows support that?
> 
> It seems that Windows doesn't support unsetenv:
> 
>   posixwin.o: In function `f_4452':
>   c:\src\chicken-core/posixwin.c:7400: undefined reference to `unsetenv'

OK, thanks, I've pushed this.  I've also added a small note in the
sources, because that was nonobvious to me :)

Should this one go into stability?  I'm inclined to say "yes".

Cheers,
Peter
-- 
http://www.more-magic.net



reply via email to

[Prev in Thread] Current Thread [Next in Thread]