This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA/mingw32] environment variables are case-insensitive on win32
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Jerome Guitton <guitton at adacore dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Fri, 07 Dec 2012 11:29:19 +0200
- Subject: Re: [RFA/mingw32] environment variables are case-insensitive on win32
- References: <1354282597-22691-1-git-send-email-guitton@adacore.com> <834nk7urs0.fsf@gnu.org> <20121130140844.GF2768@adacore.com> <83y5hjt8ll.fsf@gnu.org> <20121130153401.GH2768@adacore.com> <83wqx3t6r9.fsf@gnu.org> <20121130162852.GD32262@adacore.com> <83pq2uudjj.fsf@gnu.org> <20121207092202.GJ7555@adacore.com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Fri, 7 Dec 2012 10:22:02 +0100
> From: Jerome Guitton <guitton@adacore.com>
> Cc: gdb-patches@sourceware.org
>
> Eli Zaretskii (eliz@gnu.org):
>
> > Configure-time test is not good enough, as the binary could then be
> > run in a different environment.
> >
> > How about making it case-insensitive for Cygwin as well?
>
> I ended up implementing Pedro's suggestion (a run-time test) but it
> does not work in the case of Cygwin; getenv/setenv acts as if we add a
> case-sensitive environment, but we use CreateProcess to create the
> inferior, so we end up with a case-insensitive environment for the
> inferior. So making it case-insensitive is indeed what makes sense.
>
> Last versions of the patch in attachment, along with the testcase.
> Fully tested on x86_64-linux; partially tested on Cygwin (e.g. without
> gdb.threads) amd mingw32 (testenv.exp only).
>
> Any comments?
The attached patch still uses a run-time test. Did you attach a wrong
patch?