hi Yang,

Just took a 2nd look at your last commit. It is one step in the right
direction, thanks for that.

However I would have appreciated that you noticed me about these
changes before so I don't waste my time by working on the patch I sent
earlier. Especially as it does one of the changes I proposed while
still missing other and still having duplicated configurations or
defines.

Thanks,

On Tue, Dec 21, 2010 at 11:01 AM, Pierre Joye <pierre.php_at_gmail.com> wrote:
> hi,
>
> It looks like there is little interest to change these files, as I'm
> certainly the 2nd person actually using them I can understand why.
>
> Maybe I should simply write a new set of makefiles and maintain them
> separetely, so you can keep the current system while I maintain the
> new ones. That would definitively be easier for me than fixing
> conflicts with cosmetic changes like those committed recently :).
>
> Cheers,
>
> On Sun, Dec 19, 2010 at 7:56 PM, Pierre Joye <pierre.php_at_gmail.com> wrote:
>> hi,
>>
>> I updated the patch to add support for curl.exe (src/). In the
>> process, the following changes have done too:
>>
>> - move all common definitions/macros in a common VC file (included in
>> src/ and lib/)
>>
>> - remove duplicate .c/.o definition (will remove the .o/.c too later,
>> so no risk to forget to update one or the other)
>>
>> - build out of the source folders, now the builds happen in
>> topdir/builds/curl-configname or lib-configname
>>  that allows one to build many config in a row
>>
>> - nmake clean work with cfg or without (for curl, lib is coming)
>>
>> Next step is to finish the cleanup and add one makefile at the top
>> directory, it will be then be possible to get all builds in one go.
>>
>> On Fri, Dec 17, 2010 at 7:33 PM, Pierre Joye <pierre.php_at_gmail.com> wrote:
>>> hi Yang,
>>>
>>> On Fri, Dec 17, 2010 at 5:32 PM, Yang Tse <yangsita_at_gmail.com> wrote:
>>>
>>>> only modifying lib/Makefile.vc6 is not good enough
>>>> for general use. If you wish to improve current VC6 Makefiles with
>>>> this, please provide a separate patch for this feature.
>>>
>>> Ah right, thanks, I only use libcurl, will update it accordingly.
>>
>> Cheers,
>> --
>> Pierre
>>
>> @pierrejoye | http://blog.thepimp.net | http://www.libgd.org
>>
>
>
>
> --
> Pierre
>
> @pierrejoye | http://blog.thepimp.net | http://www.libgd.org
>

-- 
Pierre
@pierrejoye | http://blog.thepimp.net | http://www.libgd.org
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html