Mailing-List: contact cygwin-apps-help AT cygwin DOT com; run by ezmlm
Sender: cygwin-apps-owner AT cygwin DOT com
List-Subscribe: <mailto:cygwin-apps-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-apps/>
List-Post: <mailto:cygwin-apps AT cygwin DOT com>
List-Help: <mailto:cygwin-apps-help AT cygwin DOT com>, <http://sources.redhat.com/lists.html#faqs>
Delivered-To: mailing list cygwin-apps AT cygwin DOT com
To: "Robert Collins" <robert DOT collins AT itdomain DOT com DOT au>
Cc: "Jan Nieuwenhuizen" <janneke AT gnu DOT org>, <cygwin-apps AT cygwin DOT com>,
   <cygwin-patches AT cygwin DOT com>
Subject: Re: experimental texmf packages
References: <m3bshtmxhb DOT fsf AT appel DOT lilypond DOT org>
	<878764062 DOT 20011128173421 AT nyckelpiga DOT de>
	<m37ks9lgxi DOT fsf AT appel DOT lilypond DOT org>
	<4434079433 DOT 20011129221637 AT familiehaase DOT de>
	<m3oflgy98n DOT fsf AT appel DOT lilypond DOT org>
	<9517228633 DOT 20011203135833 AT familiehaase DOT de>
	<m3lmgkwgeu DOT fsf AT appel DOT lilypond DOT org> <3C0D8535 DOT D67735D1 AT ece DOT gatech DOT edu>
	<m33d2pam3l DOT fsf AT appel DOT lilypond DOT org>
	<00d501c17d93$1936c990$0200a8c0 AT lifelesswks>
	<m3zo4x7obb DOT fsf AT appel DOT lilypond DOT org>
	<m38zcdssxd DOT fsf AT appel DOT lilypond DOT org>
	<01a801c18036$3d447350$0200a8c0 AT lifelesswks>
Organization: Jan at Appel
From: Jan Nieuwenhuizen <janneke AT gnu DOT org>
Date: 09 Dec 2001 00:21:00 +0100
In-Reply-To: <01a801c18036$3d447350$0200a8c0@lifelesswks> ("Robert Collins"'s message of "Sun, 9 Dec 2001 09:18:16 +1100")
Message-ID: <m3itbhqowz.fsf@appel.lilypond.org>
Lines: 101
User-Agent: Gnus/5.090003 (Oort Gnus v0.03) Emacs/21.1
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii

"Robert Collins" <robert DOT collins AT itdomain DOT com DOT au> writes:

> Follow to cygwin-patches please.

(oops, let's see if I can post there?)

> Thanks for freshening this up. There's still a little more to do, but it
> looks good nonetheless.

Ok.  Last status was: discussion document (otherwise it would have
gone in :-)

> > * Makefile.in (CFLAGS): Remove -Werror to allow build.
> 
> The Werror is in by design. If your patch won't build with it, I won't
> accept you patch.

Ah, good.  But it was not for my patch!  The latest cvs setup.exe
won't build without it, in my environment.  I get warnings (-> errors)
like:

/home/fred/usr/src/cygwin/cygwin-1.3.6/usr/src/cygwin/src/winsup/w32api/include/rpcndr.h:252: warning: function declaration isn't a prototype
In file included from /home/fred/usr/src/cygwin/cygwin-1.3.6/usr/src/cygwin/src/winsup/w32api/include/ole2.h:5,
                 from /home/fred/usr/src/cygwin/cygwin-1.3.6/usr/src/cygwin/src/winsup/w32api/include/shlobj.h:8,
                 from mklink2.c:3:
/home/fred/usr/src/cygwin/cygwin-1.3.6/usr/src/cygwin/src/winsup/w32api/include/objbase.h:165: warning: function declaration isn't a prototype
/home/fred/usr/src/cygwin/cygwin-1.3.6/usr/src/cygwin/src/winsup/w32api/include/objbase.h:166: warning: function declaration isn't a prototype
make: *** [mklink2.o] Error 1


> > * configure.in (LIB_AC_PROG_CXX): Bugfix for CXXFLAGS override.
> 
> What's wrong with the current method?

At some point, it feeds $(CXXFLAGS) to the shell, which complains that
CXXFLAGS is not a command.

> > * desktop.cc (etc_profile): Remove line breaks and spaces from PS1.
> 
> The line is now > 80 chars, and indent will break it up again. Is there
> some reason for this change?

Ah, that's how this bug got in, probably.  You may not notice it
because your /etc/profile won't be overwritten, or you have your own
PS1, but the default prompt from unpatched install looks like this:

    tineke AT DOOS ~
    $ logout
    Connection to doos closed.

with the fix, it looks like this:

    tineke AT DOOS ~$ logout

(but we'll have to do another sane line break, because of indent.)


> > * package_meta.cc (try_run_script): New function.
> 
> This doesn't belong here. It's nothing to do with the package, but with
> interfacing with the shell/scripts. Also (this one is minor/optional),
> cygpath and _access are deprecated - foo = io_stream::open (concat
> ("cygfile://", dir, fname, 0), "rb") followed by run_script (foo) would
> be the more OO approach here.
> 
> In the future I think we'll want a script or shell class to encapsulate
> all of this.

Ok.  In short: I tried to update the patch with a minimal amount of
extra/moved code.  The comments suggest that pkg managent is still in
flux.  I can fix the file exists check, but where do you want the
function to go?

Btw: is someone working on conflicts: or more versatile requires:?  I
may want to have a look at adding versioned and/or alternative
requires.  It would be easiest to change the syntax to something like:

   requires: foo, bar | baz >= 1.3, bla

Would that be feasible, or will the space as separator stay, and must
we go for something like:

   requires: foo bar|baz>=1.3 bla

It's a quite small patch, if the functionality is ok, I guess I/we can
fix these details?

> You're missing postinstall.h in your changelog.

Ok.

> Also postinstall.h won't compile if it's the only include

Ok.

Greetings,
Jan.

-- 
Jan Nieuwenhuizen <janneke AT gnu DOT org> | GNU LilyPond - The music typesetter
http://www.xs4all.nl/~jantien       | http://www.lilypond.org