This is the mail archive of the 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: Proposal: naim

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
>  -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-
>  -rw-r--r-- 37556/544    152870 2003-06-20 12:12:58 naim-
>  lrwxrwxrwx 37556/544         0 2003-06-20 12:13:26 naim- -> naim-

I'm actually reviewing
but most of the points are the same.

> setup.hint:
>  # 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.

> naim-
>  -rw-r--r-- 37556/544    474586 2003-06-20 11:59:16 naim-
>  -rw-r--r-- 37556/544         0 2003-06-20 12:13:06 naim-
>  -rw-r--r-- 37556/544        38 2003-06-20 12:13:06
>  (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- 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.

> naim-
>  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- is just a cat
of the manpage, but I don't think this is a problem.

> usr/doc/Cygwin/naim-
> [snip]
>  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

> [snip]
>  Build requirements:
>    libncurses
>    libncurses-devel
A general question: is there a way to specify this, and if so, how?

>  Maintainer:
>    Daniel Reed <>

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.
      |\      _,,,---,,_
ZZZzz /,`.-'`'    -.  ;-;;,_
     |,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

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