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 <[email protected]> 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 <[email protected]> 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 <[email protected]> wrote: >>> hi Yang, >>> >>> On Fri, Dec 17, 2010 at 5:32 PM, Yang Tse <[email protected]> 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
