Re: RFS: ndpmon
John R. Baskwill wrote:
> 2011/8/4 Benoît Knecht <benoit.knecht@fsfe.org>
> > [...]
> >
> > From a quick look at your package:
> >
> > - You have a debian/patches/debian-changes-1.4.0-1 patch that is
> > probably not intentional.
> >
> > - It would be great if you could use DEP-5 [1] for your
> > debian/copyright.
> >
> > [1] http://dep.debian.net/deps/dep5/
> >
> > Also, if you run 'licensecheck -r .' in the root directory of your
> > package, you'll notice that some files lack a copyright header. You
> > should probably contact upstream about that and make sure that all
> > the files are indeed released under the LGPL.
> >
> > - 'lintian -I --pedantic ndpmon_1.4.0-2_*.changes' had this to say:
> >
> > W: ndpmon source: configure-generated-file-in-source config.status
> > W: ndpmon source: configure-generated-file-in-source config.cache
> > W: ndpmon source: configure-generated-file-in-source config.log
> > W: ndpmon source: out-of-date-standards-version 3.9.1 (current is
> > 3.9.2)
> > I: ndpmon source: debian-watch-contains-dh_make-template
> > I: ndpmon: spelling-error-in-binary usr/sbin/ndpmon Recieved Received
> > I: ndpmon: spelling-error-in-binary usr/sbin/ndpmon adress address
> > I: ndpmon: spelling-error-in-binary usr/sbin/ndpmon unkown unknown
> > I: ndpmon: spelling-error-in-binary usr/sbin/ndpmon unkown unknown
> > I: ndpmon: spelling-error-in-binary usr/src/ndpmon/ndpmon.o Recieved
> > Received
> > I: ndpmon: spelling-error-in-binary usr/src/ndpmon/ndpmon.o adress
> > address
> > I: ndpmon: spelling-error-in-binary usr/src/ndpmon/neighbors.o unkown
> > unknown
> > I: ndpmon: spelling-error-in-binary usr/src/ndpmon/neighbors.o unkown
> > unknown
> > E: ndpmon: helper-templates-in-copyright
> > I: ndpmon: spelling-error-in-manpage usr/share/man/man8/ndpmon.8.gz
> > allows to allows one to
> >
> > (Run it with '-i' to get a detailed explanation for each warning or
> > error.)
> >
> > - Your .deb contains plenty of .{c,h,o} files, and the full source in
> > /usr/src; I don't think it's what you intended to do, is it?
> >
> > - In debian/control, the description of the package contains
> > information about when, where and by whom the software was
> > developed; I don't think it's relevant here. You also depend
> > explicitly on some libraries, but these should be in
> > ${shlibs:Depends} already if the package links against them.
> >
> > I hope this helps. Don't hesitate to ask if you have problems/questions
> > about these issues.
>
> Thank you very much for taking the time to look at my package. I will work
> on the items you listed.
Great! A couple more things, in case you have too much time on your
hands :)
- debian/docs: You shouldn't install CHANGES, FILES, MD5SUMS or
VERSION; they're really not useful to the user (except changes, but
it's installed as changelog.gz by dh_installchangelogs already).
- debian/ndpmon.init: Instead of hardcoding variables such as INIT or
LOGDIR, you could source /etc/default/ndpmon (you'd have to create
it in your package) so that users can easily change these paths.
Also, and correct me if I'm wrong, but isn't $"string" a bashism?
- debian/{postinst,prerm,postrm} do not do anything, you should remove
them. I also think you can safely remove debian/preinst; the daemon
will be stopped on upgrades by the prerm script generated by
debhelper.
Cheers,
--
Benoît Knecht
Reply to: