This is the mail archive of the
mailing list for the Cygwin project.
Re: Proposal: naim
- From: Igor Pechtchanski <pechtcha at cs dot nyu dot edu>
- To: Daniel Reed <n at n dot ml dot org>
- Cc: cygwin-apps at cygwin dot com
- Date: Fri, 11 Jul 2003 10:25:01 -0400 (EDT)
- Subject: Re: Proposal: naim
- Reply-to: cygwin-apps at cygwin dot com
On Wed, 25 Jun 2003, Daniel Reed wrote:
> Heya, I'm the maintainer of a multi-protocol console chat client called
> naim. I've been in touch with quite a few Cygwin users using naim, and
> thought I'd see about getting naim "into" Cygwin for their convenience.
> ircII already appears to be included, though there is no "Communication"
> category (ircII is simply listed in Net). I've listed Communication as a
> category in my setup.hint, but I will adjust the packaging procedure to
> simply list Net if that is more appropriate. naim documentation is available
> from http://naim.n.ml.org/.
> -rw-r--r-- 37556/544 220 2003-06-20 12:13:24 setup.hint
> -rw-r--r-- 37556/544 476934 2003-06-20 12:13:22 naim-0.11.5.9.cyg7-1-src.tar.bz2
> -rw-r--r-- 37556/544 152870 2003-06-20 12:12:58 naim-0.11.5.9.cyg7-i686-pc-cygwin.tbz2
> lrwxrwxrwx 37556/544 0 2003-06-20 12:13:26 naim-0.11.5.9.cyg7-1.tar.bz2 -> naim-0.11.5.9.cyg7-i686-pc-cygwin.tbz2
I'm actually reviewing
but most of the points are the same.
> # This file is generated by make cyg, do not edit directly.
> @ naim
> sdesc: "Console AIM, ICQ, IRC, and Lily client"
> ldesc: "Console AIM, ICQ, IRC, and Lily client"
> requires: libncurses6 cygwin
Shouldn't this be libncurses7 already?
> category: Communication Net
I don't know if creation of new categories is approved... At a guess,
"Net" is the same as "Communication", anyway.
> -rw-r--r-- 37556/544 474586 2003-06-20 11:59:16 naim-0.11.5.9.cyg7.tar.gz
> -rw-r--r-- 37556/544 0 2003-06-20 12:13:06 naim-0.11.5.9.cyg7-1.patch
> -rw-r--r-- 37556/544 38 2003-06-20 12:13:06 naim-0.11.5.9.cyg7-1.sh
> (no patches for Cygwin, just ./configure --prefix=/usr && make install-strip)
The .sh script should also extract the tarball into a directory with the
name of the package (i.e., naim-0.11.5.9.cyg10/) and (technically) apply
the patch. I realize that the patch is currently null, so I'm not sure
the second step is all that important. The first one is necessary,
however. The patch also normally contains the Cygwin readme, but here the
package is Cygwin-aware, so I don't know if that applies.
> drwxr-xr-x 37556/544 0 2003-06-20 12:00:10 ./
> drwxr-xr-x 37556/544 0 2003-06-20 12:07:24 ./usr/
> drwxr-xr-x 37556/544 0 2003-06-20 12:07:24 ./usr/bin/
The leading ./ is gone in the new package, plus some symlinks were added.
All seems in order. The usr/doc/naim-0.11.5.9.cyg10/README is just a cat
of the manpage, but I don't think this is a problem.
> Packaging procedure:
> To create a Slackware pkgtool-compatible tarball, simply run:
> ./configure \
> '--prefix=/usr' \
> 'CFLAGS=-O20 -funroll-loops -fexpensive-optimizations' \
> && make tgz
> If configure is run under Cygwin, make tgz will adjust itself to conform to
> Cygwin standards. Use make cyg instead of make tgz to produce a complete
> (source and binary) Cygwin release.
What will be the version number the name of the tarball produced by the
> Build requirements:
A general question: is there a way to specify this, and if so, how?
> Daniel Reed <email@example.com>
Other than the minor nits for the setup.hint (libncurses7) and the patch
(README), the only problem is the build script in the source package.
Once that is fixed, I think the packaging will be ok. I can't do a
functionality review, as I don't use the above messengers.
|\ _,,,---,,_ firstname.lastname@example.org
ZZZzz /,`.-'`' -. ;-;;,_ email@example.com
|,4- ) )-,_. ,\ ( `'-' Igor Pechtchanski, Ph.D.
'---''(_/--' `-'\_) fL a.k.a JaguaR-R-R-r-r-r-.-.-. Meow!
"I have since come to realize that being between your mentor and his route
to the bathroom is a major career booster." -- Patrick Naughton