[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] [cygwin|mingw] cwrapper cleanups.
From: |
Charles Wilson |
Subject: |
Re: [PATCH] [cygwin|mingw] cwrapper cleanups. |
Date: |
Sat, 24 Jan 2009 11:47:25 -0500 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.8.1.19) Gecko/20081209 Thunderbird/2.0.0.19 Mnenhy/0.7.5.666 |
Ralf Wildenhues wrote:
> * Charles Wilson wrote on Wed, Jan 21, 2009 at 10:14:22PM CET:
>> libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src)
>> [file scope]: re-organized includes and portability
>> macros. Avoid oldnames on MINGW32 and MSVC for
>> setmode/stat/chmod/getcwd/putenv. Declare _putenv on
>> MINGW32 when -ansi.
>
> OK, thanks.
Pushed.
> The only part that looks arbitrary is this one:
>
>> Use namespaced macro LT_DEBUGWRAPPER.
>
> It is not accompanied by any documentation change or testsuite change.
> This of course isn't this patch's fault, but shows that there is some
> documentation and some test missing. It would be nice if this were
> addressed in another patch. :-)
Preparing a followup patch (#1).
> If you want to add a round using -Wall to the cwrapper test (or add
> -Wall to one of the current rounds), consider such a change preapproved.
Preparing a followup patch (#2).
--
Chuck