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

libtenjin-perl package on mentor.d.n



Hi Harlan,

thanks for packaging libtenjin-perl Perl module.
If you are interested in packaging Perl modules, please consider to join
our team requesting an alioth account [1]

If you need help or you have any questions don't hesitate to ask on
mailing list or on IRC (you can find our funny 'faces' on #debian-perl,
OFTC network :-))

Below my comments on your package: 
Just a note before starting: I'm not a DD, so feel free to ignore my
review.

* debian/control

   + Build-Depends: debhelper (>= 7): you can update it to (>= 8)
   + Standards-Version: 3.9.1: you can update it to 3.9.2
   + Missing Vcs-* fields: we use git repository for all our packages.
     For instruction on how to use it, please read 
     http://pkg-perl.alioth.debian.org/git.html
   + long description: please substitute 'Tenjin' with 'libtenjin-perl'
   + short description: should omit any initial indefinite or 
     definite article like 'a', 'an' etc. etc.

* debian/compat

   + Update it to 8

* debian/libtenjin-perl.docs

   + if the README file does not contains useful information, 
     usually, we don't release it. In this case I think we can 
     release, but wait for a DD opinion.
   + Perhaps Changes file can be released as 'upstream changelog' 
     but also in this case, wait for a DD opinion

* debian/copyright
  
   + use DEP-5 format as below

      Format-Specification: 
      Maintainer: 
      Source: 
      Name: 

* debian/changelog

  + please close an ITP/RFP bug
    http://pkg-perl.alioth.debian.org/howto/RFP.html 


* debian/patches

  + Use DEP-3 headers:

     Description: <what the patch does>
     Author: <your name> <your email>
     Forwarded: <no, if there is no reason to forward to upstream>
     Last-Update: <date of last refresh>

Building the package I get this lintian messages:

W: libtenjin-perl source: out-of-date-standards-version 3.9.1 (current is 3.9.2)
(looks my previous note on debian/control)

W: libtenjin-perl: description-synopsis-starts-with-article
(looks my previous note on debian/control)

I need to check the dependencies you specified in d/control: I'll do it
next time, when all this things will be fixed.

..and finally, I hope to see you soon in our team :-)

Cheers,
Fabrizio.

[1]http://wiki.debian.org/Teams/DebianPerlGroup


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


Reply to: