This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFC] Add support for locally modified environment variables for windows-nat.c


On Apr 20 12:26, Pierre Muller wrote:
> A while ago
> Sebastian Puebla submitted a patch for 
> support of locally modified environment variables.
> 
> http://sourceware.org/ml/gdb-patches/2010-05/msg00317.html
> http://sourceware.org/ml/gdb-patches/2010-08/msg00026.html
> http://sourceware.org/ml/gdb-patches/2010-11/msg00128.html
> 
> Nevertheless, I suspect that his patch would not have worked for Cygwin,
> because Cygwin converts several environment variables
> from windows style to POSIX style.

Do you suspect or did you actually test it?  What you're trying to
accomplish is actually the task of the 

  cygwin_internal (CW_SYNC_WINENV);

call right before calling CreateProcess.  This allows to keep the actual
underlying details of the environment handling entirely in the hands of
Cygwin.  Especially copying an internal list of env vars from Cygwin
into GDB makes me very uncomfortable since this is bound to break as
soon as we decide to handle things a bit different in Cygwin.

Of course it's cgf's call, but I don't think this is the right thing to
do.  If this is actually broken, it's the cygwin_internal(CW_SYNC_WINENV)
implementation in Cygwin which should be fixed, rather than GDB going
out of its way to emulate what happens in Cygwin internally.


Corinna

-- 
Corinna Vinschen
Cygwin Project Co-Leader
Red Hat


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]