[libspreadsheet-readsxc-perl] 01/01: add TODO items to d/changelog

[gregor herrmann]

This is an automated email from the git hooks/post-receive script.

gregoa pushed a commit to branch master
in repository libspreadsheet-readsxc-perl.

commit cede98f34439f0910c5b64074de385866b7c9745
Author: gregor herrmann <gregoa at debian.org>
Date:   Wed Dec 20 19:17:47 2017 +0100

    add TODO items to d/changelog
    
    Gbp-Dch: Ignore
---
 debian/changelog | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index f924762..4951e1e 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,5 +1,41 @@
 libspreadsheet-readsxc-perl (0.20-1) UNRELEASED; urgency=low
 
+  TODO:
+  - debian/watch missing (dh-make-perl creates a correct one)
+    (oh, and it was in git before the upstream merge)
+  - debian/copyright:
+    + upstream is licensed as "same as perl" which translates to
+      "Artistic or GPL-1+"
+    + typically we use "stand-alone" license paragraphs
+    + licensing debian/* under a different license than upstream is possible
+      but unusal and not recommended (as it makes forwarding patches
+      difficult)
+    + dh-make-perl provies a not-so-bad template for debian/copyright
+  - debian/control:
+    + Standards-Version: 4.1.2 (dh-make-perl ...)
+    + short description should be a noun phrase, i.e. work in a sentence like
+      <package> is (a) <short description>
+      an easy trick is to say "module to <do something>"
+    + I'd shorten the long description a bit before it gets to technical,
+      maybe after "respective spreadsheets."
+    + "Testsuite: autopkgtest-pkg-perl" missing (dh-make-perl ...)
+      this also needs debian/tests/pkg-perl/smoke-files with
+        t/
+        t.sxc
+      (for details: https://pkg-perl.alioth.debian.org/autopkgtest.html )
+    + Vcs-Git: https://anonscm.debian.org/git/pkg-perl/packages/libspreadsheet-readsxc-perl.git
+    + Vcs-Browser: https://anonscm.debian.org/cgit/pkg-perl/packages/libspreadsheet-readsxc-perl.git
+    + missing: Build-Depends-Indep: libarchive-zip-perl, libxml-parser-perl
+      (dh-make-perl ...), otherwise the tests fails in a chroot / when the
+      packages are not installed. (building with cowbuilder or sbuild helps)
+  - lintian says:
+    W: libspreadsheet-readsxc-perl: spelling-error-in-changelog unusal unusual
+    usually we would create a patch and forward it upstream
+  - d/changelog: s/UNRELEASED/unstable/ to move the package in PET to the
+    "ready for upload" section which triggers (another) review and upload:
+    https://pet.debian.net/pkg-perl/pet.cgi
+  - `dh-make-perl --refresh' will help with _some_ of the above steps
+
   * Initial release (Closes: #884866).
 
  -- Andrius Merkys <andrius.merkys at gmail.com>  Fri, 08 Dec 2017 11:13:22 +0200

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-perl/packages/libspreadsheet-readsxc-perl.git