Alle domenica 2 ottobre 2011, Praveen A ha scritto:
> Someone please review the changes and upload. All changes
> are in git repo.

Some notes (in random order) about the packaging:
- why the usage of postinst and prerm to create the symlink for
  jquery.js? isn't a .links file for dh_link (hence a static symlink)
  enough for this job?
- in all the packages descriptions and in the doc-base abstract there is
  "and import their menus over DBus.", where this "DBus" occurrency
  should be "D-Bus"
- copyright misses the current year (2011) for your debian/* changes
- in the doc-base file, Files should most probably be something like
  Files: /usr/share/doc/libdbusmenu-qt-doc/*.html
         /usr/share/doc/libdbusmenu-qt-doc/search/*.html
  (given it represents all the HTML files of the documentation)
- what's the point of a symbols file, if you set a strict shlibs 
  manually with dh_makeshlibs? Either drop the strict shlibs, or the
  symbols file
- most probably you can avoid 001-dbusmenu-doc-path.diff at all, and
  just move the files to the right place using a
  libdbusmenu-qt-doc.install like:
  usr/share/doc/dbusmenu-qt/ usr/share/doc/libdbusmenu-qt-doc
  (this would avoid a (most probably) unnecessary patch)
- the libdbusmenu-qt2 description talks about "shared libraries", while
  there is actually just one
- in the libdbusmenu-qt-doc description, "html" should be properly
  all uppercase
- given you need an upload to NEW, why not use the opportunity to add a
  libdbusmenu-qt-dbg debug package?

-- 
Pino Toscano

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

-- 
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-kde-talk

Reply via email to