This is the mail archive of the
cygwin-apps@cygwin.com
mailing list for the Cygwin project.
Re: experimental texmf packages
- From: Jan Nieuwenhuizen <janneke at gnu dot org>
- 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>
- Date: 09 Dec 2001 00:21:00 +0100
- Subject: Re: experimental texmf packages
- Organization: Jan at Appel
- References: <m3bshtmxhb.fsf@appel.lilypond.org><878764062.20011128173421@nyckelpiga.de><m37ks9lgxi.fsf@appel.lilypond.org><4434079433.20011129221637@familiehaase.de><m3oflgy98n.fsf@appel.lilypond.org><9517228633.20011203135833@familiehaase.de><m3lmgkwgeu.fsf@appel.lilypond.org> <3C0D8535.D67735D1@ece.gatech.edu><m33d2pam3l.fsf@appel.lilypond.org><00d501c17d93$1936c990$0200a8c0@lifelesswks><m3zo4x7obb.fsf@appel.lilypond.org><m38zcdssxd.fsf@appel.lilypond.org><01a801c18036$3d447350$0200a8c0@lifelesswks>
"Robert Collins" <robert.collins@itdomain.com.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@DOOS ~
$ logout
Connection to doos closed.
with the fix, it looks like this:
tineke@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@gnu.org> | GNU LilyPond - The music typesetter
http://www.xs4all.nl/~jantien | http://www.lilypond.org