[Bug 690927] Re: Please merge tpb 0.6.4-6 (universe) from Debian unstable (main)

[Evan Broder]

Your changelog entry needs some work:

 - (small nit) Our standard form for LP closers is "(LP: #12345)" not "(Closes LP: #12345)"
 - Don't bother putting in the changelog that you're changing Maintainer/Original-Mainatainer. There's no need to document something that we will always do

But most importantly,

 - Please *do* actually list the Ubuntu changes that you're keeping.
Merging is an opportunity to reconsider the changes that have been
introduced in the past and confirm that they're still needed. Without
this, it's difficult to track the ongoing Ubuntu diff from Debian.

And, in fact, I see several places where we could shrink the Ubuntu
diff, which makes packages more maintainable in the long run. For
instance, there are some commented-out lines which were unnecessarily
removed from debian/rules, and a blank line that was removed from
debian/tpb.preinst. Also, there are a bunch of changes in the .po files
that seem superfluous.

We also discussed adding a new translation on #ubuntu-motu. Since I
don't speak Indonesian and can't review your work in that area, I'm
uncomfortable sponsoring that work myself. Since, as far as I know, we
can only use Rosetta for packages in main, I think the best thing to do
would be to submit this translation file to Debian's Indonesian l10n
list. If they approve of it, then I wouldn't have a problem including
the translation until Debian is unfrozen.

I'm going to go ahead and unsubscribe ubuntu-sponsors for now, but feel
free to resubscribe them when you've updated the patch.

** Changed in: tpb (Ubuntu)
     Assignee: Evan Broder (broder) => Mahyuddin Susanto (udienz)

** Changed in: tpb (Ubuntu)
       Status: In Progress => Confirmed

** Changed in: tpb (Ubuntu)
   Importance: Undecided => Wishlist

-- 
You received this bug notification because you are a member of Ubuntu
Sponsors Team, which is a direct subscriber.
https://bugs.launchpad.net/bugs/690927

Title:
  Please merge tpb 0.6.4-6 (universe) from Debian unstable (main)

Status in “tpb” package in Ubuntu:
  Confirmed

Bug description:
  Binary package hint: tpb

tpb (0.6.4-6) unstable; urgency=low

  * Fix pending l10n issues with new/updated Debconf translations. Thanks
    again to Christian Perrier for his tireless work on this front!
    * Spanish (Omar Campagne).  Closes: #582318
    * Danish (Joe Hansen).  Closes: #597762
    * Italian (Vincenzo Campanella).  Closes: #600220
  * Updated Standards-Version.

 -- Steve McIntyre <93sam at debian.org>  Mon, 18 Oct 2010 12:53:41 +0100

tpb (0.6.4-5) unstable; urgency=low

  * Change the priority of the udev rule so that udev doesn't change
    permissions on /dev/nvram again afterwards. Thanks to Kevin Coyner for
    a patch. Closes: #569111

 -- Steve McIntyre <93sam at debian.org>  Wed, 10 Feb 2010 12:26:57 +0000

tpb (0.6.4-4) unstable; urgency=low

  * Add a udev rules file to set up appropriate permissions on /dev/nvram.
    Thanks to Paul Martin for the patch. Closes: #551086. Also remove the
    old conf file /etc/devfs/conf.d/tpb if it still exists.
  * Remove old MAKEDEV code and dependency. Closes: #477061

 -- Steve McIntyre <93sam at debian.org>  Sun, 17 Jan 2010 21:58:47 +0000

tpb (0.6.4-3) unstable; urgency=low

  * Initial upload from new maintainer.
  * Update to recent Standards-Version and fix a load of lintian warnings:
    + Use debhelper 6 and higher, in debian/compat
    + Add misc:Depends
    + Add debhelper tokens as appropriate
    + Remove old devfs cruft
    + Add Pre-depend on debconf for preinst
  * Remove old makedev question
  * Add Russian debconf translation. Thanks to Yuri Kozlov. Closes: #541762
  * Add French translation and debconf translation. Thanks to Frédéric
    Boiteux. Closes: #448613.

 -- Steve McIntyre <93sam at debian.org>  Tue, 22 Sep 2009 23:38:41 +0100