[SCM] Packaging of Email::Find in Debian branch, master, updated. 281ecc3e0ee6fb21ebbaf6ed3607097e7c2e59bb

[gregor herrmann]

The following commit has been merged in the master branch:
commit 281ecc3e0ee6fb21ebbaf6ed3607097e7c2e59bb
Author: gregor herrmann <gregoa at debian.org>
Date:   Thu Jul 18 16:47:46 2013 +0200

    add TODOs
    
    Git-Dch: Ignore

diff --git a/debian/changelog b/debian/changelog
index d43f7fe..26ef620 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,5 +1,35 @@
 libemail-find-perl (0.10-dfsg-3) UNRELEASED; urgency=low
 
+  NOTES on procedures, in general:
+  - please s/UNRELEASED/unstable/ when you think a package is ready for
+    review, this will show it on http://pet.debian.net/pkg-perl/pet.cgi in the
+    "ready for upload section", so not only people following commit messages
+    on IRC will see it
+  - please don't tag packages, tagging should happen by the person who uploads
+    the package to the archive at upload time to ensure that the tag is the
+    same as what's in the archive
+
+  TODO:
+  - it looks like you forgot to push the tags for this package
+  - usually "+dfsg" is used for a repackaged tarball. doesn't make sense to
+    change for a -2 upload, but might be worth to change after the upload as a
+    preparation for the next one. cf. our repackaging guide:
+    http://pkg-perl.alioth.debian.org/howto/repacking.html
+    for setting up the whole machinery
+  - I don't see any text or script or anything how the repacked tarball is
+    created? at least a note in d/copyright is required, cf.
+    http://pkg-perl.alioth.debian.org/howto/repacking.html#4__copyright_file
+  - d/control: the short description should be a noun phrase, e.g. "module to
+    find ..."
+  - patch fix_pod2man_errors.diff : why is forwarding not needed? looks like a
+    perfect example of a bug that upstream should fix :)
+    please forward to the CPAN RT and add the URL to the patch
+    (the forward-patch script in our scripts repo can help)
+  - switching to the dh(1)-form of debian/rules means that the README is not
+    installed anymore. which is good, since the file is useless (text version
+    of the POD/manpage) but you should mention in d/changelog that it's not
+    shipped anymore
+
   * debian/control:
     + changed Maintainer field for perl group.
     + added Uploaders field.

-- 
Packaging of Email::Find in Debian