[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: New Package: fish

On Mon, 27 Jun 2005 22:04:16 +0200, Axel Liljencrantz wrote:

> Hello Michael,
> Thank you for your input. I'm sorry for the delay in my reply, but
> while I was writing my reply, my appendix burst, and I've spent the
> last few days recovering from surgery.

Hope everything is going well enough to take a first comment
(I'd like to give Oliver a chance to review/approve this). :-)

> >  * "License: GPL"? No. The C source files disagree. => LGPL
> All code written by me is released under the GPL.


> >  * Your %postun script will break upgrades, as it will be executed
> >    last. Look like e.g. "bash" does it.
> I run fedora myself, and I've upgraded hundreds of time over the last
> few months, without any issues. The fish entry is not removed after an
> upgrade.

It is.  %postun is executed _last_ during an upgrade operation,
i.e. _after_ the new package was installed and _after_ the new package
had added its entry to /etc/shells. Effectively, it removes the entry.

> I checked what bash does, and the only difference is that the editing
> of /etc/shells is conditioned on
> if [ "$1" = 0 ]; then
> which I don't know what it does, since I haven't been able to find out
> what parameters are sent to the postun script. Sorry, but I don't own
> any books on RPM.
> Can you elaborate on the problem?

Some information is summed up here:

Running scriptlets only in certain situations

Entry page: http://fedoraproject.org/wiki/PackagingGuidelines

A community-enhanced online version of Ed Bailey's Maximum RPM book
can be found here: http://rpm.org/max-rpm-snapshot/

> >  * Package doesn't build with $RPM_OPT_FLAGS, because it overrides CFLAGS
> >    with its own flags.
> fish uses some c99 features, I've had issues with some gcc version
> unless --std=gnu99 is specified. fno-strict-aliasing is there to
> remove warnings on Alpha. I fear that fish will not build correctly in
> all cases without gnu99.

Use of %configure fixed this issue already. Your code now inherits
the CFLAGS passed in from the outside.
> >  * (RFE: Is it possible to install documentation files into %_docdir like
> >    most other packages? i.e. %_datadir/doc/%name-%version)
> This required minor changes to the source, but the documentation
> directory is now specified by;
> %configure docdir=%_datadir/doc/%{name}-%{version}

checking for a BSD-compatible install... /usr/bin/install -c
./configure: line 2756: /usr/share/doc/fish-1.11.1: No such file or directory
checking for /proc/self/stat... yes

Is this related? It's new in release 4.
Michael Schwendt <mschwendt users sf net>
Fedora Core release 4 (Stentz) - Linux 2.6.11-1.1369_FC4
loadavg: 1.29 1.50 1.44

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]