Please merge laptop-mode-tools 1.55 (universe) from debian (unstable)

Bug #602881 reported by David Sugar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
laptop-mode-tools (Ubuntu)
Expired
Wishlist
Unassigned

Bug Description

Binary package hint: laptop-mode-tools

There are many upstream fixes that seemed worthwhile to get since the last merge.

Tags: patch
Changed in laptop-mode-tools (Ubuntu):
status: New → Confirmed
Revision history for this message
Stefano Rivera (stefanor) wrote :

Hi, you let an auto-generated quilt patch slip through again, please format it correctly.

I don't think we need that preinst file. Everyone install this will have upgraded through lucid.

Changed in laptop-mode-tools (Ubuntu):
assignee: nobody → David Sugar (dyfet)
importance: Undecided → Wishlist
status: Confirmed → Incomplete
Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

I will revise and try to get a new patch out later today.

Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

debdiff laptop-mode-tools_1.55-1.dsc laptop-mode-tools_1.55-1ubuntu1.dsc

Among other things, I removed the preinst script.

Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

debdiff laptop-mode-tools_1.52-1ubuntu2.dsc laptop-mode-tools_1.55-1ubuntu1.dsc

Changed in laptop-mode-tools (Ubuntu):
status: Incomplete → New
tags: added: patch
Revision history for this message
Benjamin Drung (bdrung) wrote :

You forgot to add ubuntu-sponsors

Revision history for this message
Stefano Rivera (stefanor) wrote :

Err that was probablay me unsubscribing them

Instead of removing "debian/patches/debian-changes-1.55-1" how about just commenting it out in debian/patches/series?

Changed in laptop-mode-tools (Ubuntu):
status: New → Incomplete
Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :
Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :
Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

new debdiff's posted

Changed in laptop-mode-tools (Ubuntu):
status: Incomplete → New
Revision history for this message
Stefano Rivera (stefanor) wrote :

This is a little tricky to review because the changelog is incomplete. We prefer a changelog entry that clearly describes the merge i.e. all remaining changes.

Some other issues I can see:
* The maintainer needs to be updated. update-maintainer knows what to do.
* It looks like debian renamed sleep.d/99laptop-mode to 01laptop-mode yet we recreate 99

Sorry, but a full review is going to need a better changelog

Changed in laptop-mode-tools (Ubuntu):
status: New → Incomplete
Revision history for this message
Benjamin Drung (bdrung) wrote :

Please resubscribe ubuntu-sponsors once you have addressed the issues mentioned in the previous comment.

Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

This should be targeted for Natty

Revision history for this message
David Sugar (dyfet-deactivatedaccount) wrote :

It will happen then when we resync....should I keep this open to verify the merge for Natty or mark this as invalid?

Revision history for this message
Stefano Rivera (stefanor) wrote :

> It will happen then when we resync

We have ubuntu modifications in laptop-mode-tools, so it won't automatically be synced. If you can please address my previous comments, target the upload at natty, and resubscribe sponsors, it can be uploaded.

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

[Expired for laptop-mode-tools (Ubuntu) because there has been no activity for 60 days.]

Changed in laptop-mode-tools (Ubuntu):
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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