This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: RFC: setup.exe, mingw-libgpg-error/mingw-gcrypt
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