Ubuntu
powersave package

Please merge powersave 0.15.20-6(universe) from debian unstable(main)

Bug #395640 reported by Bhavani Shankar
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
powersave (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Debian has a new version to be merged

powersave (0.15.20-6) unstable; urgency=low

  * debian/copyright
    - Reference the versioned GPL-2 file instead of the GPL symlink.
  * debian/control
    - Bump Standards-Version to 3.8.2. No further changes.
    - Bump Build-Depends on debhelper to (>= 7).
    - Bump Depends on lsb-base to (>= 3.2-14) for status_of_proc.
  * debian/compat
    - Bump debhelper compat level to 7.
  * debian/powersaved.preinst
    - Use "set -e" instead of "#!/bin/sh -e".
  * debian/rules
    - Include debhelper.mk before autotools.mk as recommended by the cdbs
      documentation.
    - Drop DEB_SHLIBS_INCLUDE, no longer necessary.
  * debian/powersaved.init
    - Use global PIDFILE variable.
    - Add "status" action using status_of_proc from lsb-base.

 -- Michael Biebl <email address hidden> Wed, 01 Jul 2009 18:56:37 +0200

Related branches

lp:ubuntu/karmic/powersave
Revision history for this message
Bhavani Shankar (bhavi) wrote :
Changed in powersave (Ubuntu):
status: New → Confirmed
Revision history for this message
Nick Ellery (nick.ellery) wrote :

Your patch includes dropping misc:Depends on two packages, but I don't see where this is mentioned in debian/changelog. Please investigate this change and see if it's still necessary to keep.

Changed in powersave (Ubuntu):
status: Confirmed → Incomplete
Revision history for this message
Bhavani Shankar (bhavi) wrote :

That was a change carried over by previous 2 versions

please see:

https://bugs.launchpad.net/ubuntu/+source/powersave/+bug/326114

and I ve removed it as its not necessary in new version

attached is the new diff

regards

Changed in powersave (Ubuntu):
status: Incomplete → Confirmed
Revision history for this message
Nick Ellery (nick.ellery) wrote :

What justifies it being removed? In the bug you provided, was the change carried forward, or made alongside the merge (if so, for what reason)?

Revision history for this message
Bhavani Shankar (bhavi) wrote :

Ok again referring to

https://bugs.launchpad.net/ubuntu/+source/powersave/+bug/312846

it was a lintian override

and here is the fixed detailed description

Regards

Revision history for this message
Nick Ellery (nick.ellery) wrote :

I'm not sure I understand what lintian warning this fixes. From what I can see dropping the misc:Depends is causing lintian errors, not fixing them:

$ lintian powersave_0.15.20-6ubuntu1_source.changes
W: powersave source: debhelper-but-no-misc-depends libpowersave11
W: powersave source: debhelper-but-no-misc-depends libpowersave-dev

It's a bit confusing when you've referenced two different bugs, neither of which obviously justify the matter at hand. Perhaps I'm just in need of a better explanation.

Also in the future, please be more specific in your changelog entries and actually mention what lintian error you are fixing.

Revision history for this message
Bhavani Shankar (bhavi) wrote :

Thats the reason I stated above, perhaps my english needs a bit polishing I presume.. That error was overriden by dropping that change and keeping the debian one as it is .. the diff the 2nd one what I have attached is right one I believe ...

Regards

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package powersave - 0.15.20-6ubuntu1

---------------
powersave (0.15.20-6ubuntu1) karmic; urgency=low

  * Merge from debian unstable, remaining changes: LP: #395640
    - Add Replaces/Provides for powernowd and apmd, to allow installation
      with *ubuntu-desktop meta packages.
    - Drop changes for powerdev group used for dbus access on Debian:
      + Drop debian/patches/man_powerdev_group.patch.
      + Drop debian/patches/fix_script_message_handling.patch
      + Drop debian/powersaved.postinst.
      + Do not depend on adduser package.
    - Remove (0.12.0-1) from debian/NEWS which doesn't apply to Ubuntu.
    - Remove debian/README.Debian which doesn't apply to Ubuntu as well.

 -- Bhavani Shankar <email address hidden> Sun, 05 Jul 2009 05:56:05 +0530

Changed in powersave (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.