This is the mail archive of the cygwin-apps mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]