On 3/14/2010 5:01 PM, Dave Korn wrote: > On 14/03/2010 18:35, Christopher Faylor wrote: >> This has always bugged me too. > > Well, we could do this: > > Index: Makefile.am > =================================================================== > RCS file: /cvs/cygwin-apps/setup/Makefile.am,v > retrieving revision 2.77 > diff -p -u -r2.77 Makefile.am > --- Makefile.am 28 Jun 2009 03:50:42 -0000 2.77 > +++ Makefile.am 14 Mar 2010 20:29:05 -0000 > @@ -321,4 +321,4 @@ snapshot: release > > .PHONY: ${INST_SUBDIRS} > ${INST_SUBDIRS}: > - ${MAKE} -C $@ install > + ${MAKE} -C $@ -q || ${MAKE} -C $@ install > > ... which only does the install if there would be anything to do during the > 'all' part of the build, but that isn't robust against deleting all or part of > the $objdir/inst tree. It does speed up the build very nicely though! We > could even add just a few extra static dependencies that would re-run the > install stage(s) if the needed lib(s) wasn't found in the inst/ tree. (Or we > could even go back to linking the build libs directly out of the > sub-$objdirs.)
I've got a build working here with mingw-libgpg-error and mingw-libgcrypt, which completely ignores those two subdirs in the setup source tree. The necessary changes to configure.ac/Makefile.am were surprisingly simple. However, the build process for setup still always recurses into libgetopt++/, but that doesn't annoy me QUITE as much. Well, ideally you still need /something/ to avoid that -- because I don't plan to ITP mingw-libgetopt++ (sorry). >> Moving it out of the tree entirely would be a nicer fix. > > Sounds good too. I checked the whole lot in in the first place because we > were in a rush to get it done, and because the thought of making it into > separate packages for the distro didn't occur to me at the time. There's > certainly no special reason for it. Sure, we were under the gun then for a vulnerability. But...now we're not, and it'd be nice to clean that up some (hey, and update to a newer version while we're at it...) So, I'm going to post the following: ITP: mingw-libgpg-error ITA: libgpg-error (*) ITP: mingw-libgcrypt (**) ITA: libgcrypt (*) (**) Once those are all in the distro, then I'll post the necessary patch to cygwin-patches. After a few weeks to simmer, then we can talk about a wholesale 'cvs rm'... (*) Gergely Budai is MIA even if these packages aren't currently, officially, listed as unmaintained. Plus, because of some shared/conflicting files between the mingw- and cygwin- packages, I need to ensure that both are kept in sync, like I have done for the zlib, bz2, and xz pairs. Since the current cygwin ones are so old...either Gergely needs to adopt my new cygports, or I need to adopt his packages. Well, these two anyway. (**) It's too late tonight. I'll post these packages tomorrow. -- Chuck