Please merge usbmount 0.0.18 (universe) from Debian (testing)

Bug #418296 reported by Rogério Theodoro de Brito
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
usbmount (Ubuntu)
Fix Released
Wishlist
Andres Rodriguez

Bug Description

Binary package hint: usbmount

Hi,

My package usbmount 0.0.17.1 has a very important bugfix in Debian that fixes a race condition in the version 0.0.17 that can, under some circumstances, lead to data loss.

Please, update the package ASAP.

Thanks, Rogério Brito.

 usbmount (0.0.18) unstable; urgency=low
 .
   * usbmount:
     + migrate from vol_id to blkid; patch adapted from gregoa.
       (Closes: #548627)
     + support mounting by UUID; patch adapted from Patrick Schoenfeld.
       (Closes: #551757)
     + include extra debugging points.
   * usbmount.rules:
     + update rules for new naming scheme of recent kernels/udev combo.
   * debian/NEWS:
     + really include the file (Closes: #548785).
   * debian/control:
     + update standards version to 3.8.3 (no changes necessary).
     + reorder fields.
     + remove uploaders; Rafael is welcome any time he wants to.
   * debian/copyright:
     + fix my e-mail address.
     + consolidate entries.
 .
 usbmount (0.0.17.1) unstable; urgency=low
 .
   * usbmount:
     + revert patch from #403209 (seems to introduce race conditions)
       (Closes: #535238)
   * usbmount.conf:
     + include ext4 so that it can be mounted automatically.
     + add MOUNTOPTIONS nodiratime. Closes: #539386. (Tks Jari Aalto)
   * debian/control:
     + include comment (allowed by policy).
     + update standards version to 3.8.2 (no changes necessary).

Related branches

Revision history for this message
Andres Rodriguez (andreserl) wrote :

Hi there.

Since there is a new version I'll be merging usbmount 0.0.18 into Ubuntu Lucid. Now, if this is important fix to SRU to Ubuntu karmic, could you please provide me with a way to reproduce a data loss?

Thanks a lot :).

Changed in usbmount (Ubuntu):
assignee: nobody → Andres E. Rodriguez Lazo (andreserl)
importance: Undecided → Wishlist
status: New → In Progress
summary: - Please sync usbmount/0.0.17.1 from Debian
+ Please merge usbmount 0.0.18 (universe) from Debian (testing)
description: updated
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package usbmount - 0.0.18ubuntu1

---------------
usbmount (0.0.18ubuntu1) lucid; urgency=low

  * Merge from debian testing (LP: #418296), remaining changes:
    - debian/rules: Copy udev rules so debhelper picks it up
    - debian/install: Do not install by hand
    - debian/preinst: Remove symlink on upgrade, and remove old rules file
      unless modified in which case rename to new filename.
    - debian/postinst: Do not create symlink
    - debian/postrm: No symlink to remove
    - Bump build-depend on debhelper to install udev rules into
      /lib/udev/rules.d, bump dependency on udev to get correct version.

usbmount (0.0.18) unstable; urgency=low

  * usbmount:
    + migrate from vol_id to blkid; patch adapted from gregoa.
      (Closes: #548627)
    + support mounting by UUID; patch adapted from Patrick Schoenfeld.
      (Closes: #551757)
    + include extra debugging points.
  * usbmount.rules:
    + update rules for new naming scheme of recent kernels/udev combo.
  * debian/NEWS:
    + really include the file (Closes: #548785).
  * debian/control:
    + update standards version to 3.8.3 (no changes necessary).
    + reorder fields.
    + remove uploaders; Rafael is welcome any time he wants to.
  * debian/copyright:
    + fix my e-mail address.
    + consolidate entries.

usbmount (0.0.17.1) unstable; urgency=low

  * usbmount:
    + revert patch from #403209 (seems to introduce race conditions)
      (Closes: #535238)
  * usbmount.conf:
    + include ext4 so that it can be mounted automatically.
    + add MOUNTOPTIONS nodiratime. Closes: #539386. (Tks Jari Aalto)
  * debian/control:
    + include comment (allowed by policy).
    + update standards version to 3.8.2 (no changes necessary).
 -- Andres Rodriguez <email address hidden> Wed, 25 Nov 2009 01:53:54 -0500

Changed in usbmount (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Rogério Theodoro de Brito (rbrito) wrote : Re: [Bug 418296] Re: Please sync usbmount/0.0.17.1 from Debian

Hi, Andres.

On Nov 25 2009, Andres E. Rodriguez Lazo wrote:
> Since there is a new version I'll be merging usbmount 0.0.18 into
> Ubuntu Lucid. Now, if this is important fix to SRU to Ubuntu karmic,
> could you please provide me with a way to reproduce a data loss?

I'm not even sure that usbmount 0.0.17 would work at all with
karmic---I'm using a standard Debian sid system here.

If karmic doesn't offer vol_id, then versions < 0.0.18 are useless and
will not work.

The data loss is described in <http://bugs.debian.org/535238>.

> - Please sync usbmount/0.0.17.1 from Debian
> + Please merge usbmount 0.0.18 (universe) from Debian (testing)

BTW, how do you change a summary/title of a bug? I looked at quite some
widgets in launchpad's interface, but didn't find anything obvious.

BTW #2, if you have upload rights, please consider updating my other
packages (hfsprogs, magicfilter, among other packages to which I made QA
uploads to Debian).

Regards, Rogério Brito.

--
Rogério Brito : rbrito@{ime.usp.br,gmail.com} : GPG key 1024D/7C2CAEB8
http://rb.doesntexist.org : Packages for LaTeX : algorithms.berlios.de
DebianQA: http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br

Revision history for this message
Rogério Theodoro de Brito (rbrito) wrote : Re: [Bug 418296] Re: Please merge usbmount 0.0.18 (universe) from Debian (testing)

On Nov 25 2009, Launchpad Bug Tracker wrote:
> This bug was fixed in the package usbmount - 0.0.18ubuntu1
>
> ---------------
> usbmount (0.0.18ubuntu1) lucid; urgency=low
>
> * Merge from debian testing (LP: #418296), remaining changes:
> - debian/rules: Copy udev rules so debhelper picks it up
> - debian/install: Do not install by hand
> - debian/preinst: Remove symlink on upgrade, and remove old rules file
> unless modified in which case rename to new filename.
> - debian/postinst: Do not create symlink
> - debian/postrm: No symlink to remove
> - Bump build-depend on debhelper to install udev rules into
> /lib/udev/rules.d, bump dependency on udev to get correct version.
(...)

I would appreciate a patch against my tree so that we can reduce the
divergence and collaborate closer.

BTW, the development happens at
http://svn.debian.org/wsvn/usbmount/usbmount/trunk/

Regards, Rogério Brito.

--
Rogério Brito : rbrito@{ime.usp.br,gmail.com} : GPG key 1024D/7C2CAEB8
http://rb.doesntexist.org : Packages for LaTeX : algorithms.berlios.de
DebianQA: http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br

Revision history for this message
Andres Rodriguez (andreserl) wrote :

Hi Rogério,

Sorry for the delay in responding to you.

Yes, karmic does not have vol_id indeed, that's a very good reason to do a backport of 0.0.18 into karmic. I'll personally work on this within the next 2 weeks.

On the other hand, next to the end of the title, there's a little yellow circle. Clic on it to change the title :).

Now (I do have upload rights to Universe components), If the packages that you've made QA of do not have an Ubuntu Delta, this will be synced automatically into lucid (if they are in Debian Testing of course). If they contain an Ubuntu Delta, I'll try to track them down and make the merges.

Our Delta for usbmount is in how usbmount.rules is installed. I believe that this is a difference between Debian and Ubuntu. I'll attacht the diff for you to take a look a it if you want.

Thanks a lot!

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.