Re: RFC: arpoison

2006-02-03 Thread Thijs Kinkhorst
Hello P-A,

> I'm trying to package arpoison 0.6 (arpoison.sf.net). So far, so good:
> everything seems to work. I uploaded it to mentors.debian.net. If any of you
> would like to have a look ..

I have, and here are some results.

* First of all, the package is signed but your key is not signed by
anyone else. That doesn't make the signing much useful. It's a very good
idea to get some signatures on your key (even if you do not want to
become a Debian Developer). See http://www.debian.org/events/keysigning
or http://biglumber.com to find people who want to sign near you.

* debian/control: The description of the package is way too terse. The
text in README would be more fit for this.

* debian/copyright: The copyright holder is incomplete; you list only
the last name, and you miss the name of the other author. You also need
(a) year(s) with that: "(c) 2005-2006 Steve Buer" for example. BTW this
information misses in the upstream source aswell, see below.
This file has very long lines; although sensible editors should be able
to handle it, it doesn't look really good in 'less' for example.

* debian/dirs: contains usr/bin and usr/sbin while you only seem to
install into usr/bin.

* debian/rules: you should remove commented out stuff, and optional
targets, you don't use. No reason to keep it.

* debian/compat, debian/control: the current recommended debhelper
version is 5, not 4.

Some notes for upstream, not packaging problems, but good to feed back
to them:
* The README file claims this is ARPoison 0.5 while it's 0.6.
* It doesn't include a changelog.
* LICENSE contains a very old FSF address
* The source files do not explicitly state their licence; this is a very
good idea to do (in case the file LICENCE gets lost somewhere); just put
it in comment to the top. An example text is at the bottom of LICENCE.
It should also contain some statement like "Copyright  ".
* It contains "lookup.c" which doesn't seem to do anything interesting
and isn't built.
* Make clean yields an error if the source is already clean. Turn the
"rm" into a "rm -f".
* It has some build warnings.


Otherwise, the package works fine. It just needs some polishing up on
the details.


Thijs




signature.asc
Description: This is a digitally signed message part


Re: RFC: arpoison

2006-02-02 Thread Lucas Wall
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 02/02/2006 03:40 PM, Pieter-Augustijn Van Malleghem wrote:

> I'm trying to package arpoison 0.6 (arpoison.sf.net). So far, so good:
> everything seems to work. I uploaded it to mentors.debian.net. If any of you
> would like to have a look ..

http://people.debian.org/~mpalmer/debian-mentors_FAQ.html

K.-

- --
Lucas Wall <[EMAIL PROTECTED]>  .''`.
Buenos Aires, Argentina: :ΓΈ :   Debian GNU/Linux
http://www.kadath.com.ar   `. `'  http://www.debian.org
PGP: 1024D/84FB46D6  `-
 5D25 528A 83AB 489B 356Ahttp://people.debian.org/~lwall
 4087 BC9B 4733 84FB 46D6mailto:[EMAIL PROTECTED]
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.2 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFD4pFLvJtHM4T7RtYRAgPvAJ9Kl4UCkk8B/1sWdfcXMfg3eUwZnwCg1InQ
sEKEsnDdfdHOvoVnMcdEyis=
=oQQA
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Re: RFC: arpoison

2006-02-02 Thread Bartosz Fenski aka fEnIo
On Thu, Feb 02, 2006 at 07:40:17PM +0100, Pieter-Augustijn Van Malleghem wrote:
> I'm trying to package arpoison 0.6 (arpoison.sf.net). So far, so good:
> everything seems to work. I uploaded it to mentors.debian.net. If any of you
> would like to have a look ..

Someone already told you in your ITP bugreport that full description of the
package is one of the requirenment that you didn't pass.

So now I'm telling you that it's also requirenment for RFC/RFS on this
list.

Please be more verbose whenever you ask for comments/sponsoring, cause not
everyone wants to google to find basic informations about your package.

regards
fEnIo

-- 
  ,''`.  Bartosz Fenski | mailto:[EMAIL PROTECTED] | pgp:0x13fefc40 | irc:fEnIo
 : :' :   32-050 Skawina - Glowackiego 3/15 - w. malopolskie - Poland
 `. `'   phone:+48602383548 | proud Debian maintainer and user
   `-  http://skawina.eu.org | jid:[EMAIL PROTECTED] | rlu:172001


signature.asc
Description: Digital signature


RFC: arpoison

2006-02-02 Thread Pieter-Augustijn Van Malleghem
Hi,

I'm trying to package arpoison 0.6 (arpoison.sf.net). So far, so good:
everything seems to work. I uploaded it to mentors.debian.net. If any of you
would like to have a look ..

sources.list:
deb-src http://mentors.debian.net/debian unstable main

apt-get source arpoison

Thanks!

Pieter-Augustijn Van Malleghem


signature.asc
Description: Digital signature