Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Jim Blandy <jimb@codesourcery.com>
To: gdb-patches@sourceware.org
Subject: Re: [RFA] win32-nat.c: Simplify generation of Windows environment
Date: Thu, 07 Dec 2006 19:48:00 -0000	[thread overview]
Message-ID: <m33b7rh4fx.fsf@codesourcery.com> (raw)
In-Reply-To: <20061207095839.GA14487@calimero.vinschen.de> (Corinna Vinschen's message of "Thu, 7 Dec 2006 10:58:39 +0100")


Corinna Vinschen <vinschen@redhat.com> writes:
> the below patch simplifies the code which translates the Cygwin
> environment into the native Windows environment.  So far this is
> done in GDB manually.  However, there's a Cygwin specific function
> call which does the same for the calling process.  Using this call
> has three advantages.
>
> - We can drop a rather big chunk of code from GDB which should be the
>   task of Cygwin anyway.
> - By using the Cygwin method of converting the environment, we take
>   care of all environment variables which have to be converted in
>   some way; not only the PATH variable, but all variables which are
>   also translated by Cygwin, thus making this process more transparent.
> - Subsequent changes in Cygwin don't require to change GDB.
>
>
> Ok to apply?

I'm very much inclined to take your advice on Cygwin-related issues,
and I love the deletion of code, but I still have some questions:

Is it really okay to call cygwin_internal?  (That's not the name I'd
expect a public, stable interface to have.)

How well will this work on older versions of Cygwin?  Will people
still be able to compile GDB against the Cygwin versions they can now?


  parent reply	other threads:[~2006-12-07 19:48 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-07  9:58 Corinna Vinschen
2006-12-07 16:35 ` Corinna Vinschen
2006-12-07 19:48 ` Jim Blandy [this message]
2006-12-08  8:57   ` Corinna Vinschen
2006-12-07 21:37 ` Eli Zaretskii
2006-12-08  0:09   ` Jim Blandy
2006-12-08  7:31     ` Eli Zaretskii
2006-12-08  9:12       ` Corinna Vinschen
2006-12-08 11:31         ` Eli Zaretskii
2006-12-08 12:05           ` Corinna Vinschen
2006-12-09  8:01       ` Jim Blandy
2006-12-08 20:43   ` Christopher Faylor
2006-12-09  9:15     ` Corinna Vinschen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m33b7rh4fx.fsf@codesourcery.com \
    --to=jimb@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox