exo-mount ignores HAL volume mount options 'sync' and 'noatime'

Bug #238702 reported by jwishnie
4
Affects Status Importance Assigned to Milestone
Exo
In Progress
Unknown
exo (Ubuntu)
Fix Released
Undecided
Lionel Le Folgoc
xubuntu-default-settings (Ubuntu)
Fix Released
Low
Lionel Le Folgoc

Bug Description

Binary package hint: libexo-0.3-0

HAL lets users set mount options for block devices in /etc/hal/fdi/policy/preferences.fdi

Among these are 'sync' and 'noatime'

When Xubuntu systems are being used in public or school labs, it is valuable to force USB drives to mount SYNC/NOATIME to insure that naive users who remove drives without explicitly ejecting them (VERY COMMON) do not lose data.

To this end, Inveneo (http://www.inveneo.org) which installs labs in rural Africa edits /etc/hal/fdi/policy/preferences.fdi to add the SYNC and NOATIME options.

Unfortunately, exo-mount (and gnome-mount, but that's a different bug) does NOT respect these values!

The attached patch updates the libexo-0.3-0_0.3.4-7ubuntu2 package to respect these values so that exo-mount will follow the rules as configured in HAL.

Revision history for this message
jwishnie (jeff-wishnie) wrote :
Changed in exo:
assignee: nobody → mrpouit
Revision history for this message
Gauvain Pocentek (gpocentek) wrote :

Wouldn't it be better to send this patch upstream? It's not really xubuntu specific.

Revision history for this message
Cody A.W. Somerville (cody-somerville) wrote :

It is going upstream but the patch he provided is against our patched version of exo.

Revision history for this message
Lionel Le Folgoc (mrpouit) wrote :

I agree with Gauvain. The issue is bigger, since exo completely ignores hal mount options (volume.policy.mount_option.*), as reported on http://bugzilla.xfce.org/show_bug.cgi?id=2891 (debian/patches/xubuntu-default-mount-options.patch is only an xubuntu-specific hack which can't be sent upstream…).

Changed in exo:
assignee: mrpouit → nobody
status: New → Triaged
Changed in exo:
status: Unknown → In Progress
Revision history for this message
jwishnie (jeff-wishnie) wrote :

Problem with submitting straight upstream is that this patches ontop of the Xubuntu default-mount-options.patch

If I submit upstream I need to either include the default-mount-options code in the upstream submission, or strip it and cause a conflict with the downstream patch.

please advise.

Revision history for this message
Gauvain Pocentek (gpocentek) wrote :

There are already patches proposed upstream, so adding one will probably not help.

Revision history for this message
Don Mahurin (dmahurin) wrote :

I would also add that the ro (read-only) option should also be supported.

The modified exo-mount patch, adding suport for 'ro', is attached.

( I prefer ro mount for flash drives. It would be nice if thunar had a "remount read-write" menu option)

Revision history for this message
Lionel Le Folgoc (mrpouit) wrote :

The incoming exo release in lucid will have support for custom mount options (in /etc/xdg/xfce4/mount.rc). I don't know yet if the default config is ok (that's why I opened a task for xubuntu-default-settings), but anyway, the support is here.

Changed in exo (Ubuntu):
status: Triaged → In Progress
assignee: nobody → Lionel Le Folgoc (mrpouit)
Changed in xubuntu-default-settings (Ubuntu):
status: New → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package exo - 0.3.105-3ubuntu1

---------------
exo (0.3.105-3ubuntu1) lucid; urgency=low

  * Merge from Debian testing, remaining Ubuntu changes:
    - debian/rules: removed mangling of helpers.rc
    - debian/control: update Vcs-* fields
    - debian/patches:
      + xubuntu-update-helpers.patch: patched libexo's firefox helper desktop
        file to use firefox-3.5 icon instead of firefox icon (LP: #417354)
      + xubuntu-expire-notifications.patch: don't use non-expiring
        notifications since we use notify-osd by default (LP: #423936)
      + series: refreshed.

  * Drop gtk doc changes, unneeded delta:
    - drop --enable-gtk-doc switch from configure
    - drop build-dep on gtk-doc-tools.

  * debian/patches: renamed 04_update_helpers.patch to
    xubuntu-update-helpers.patch as it is xubuntu-specific.

  * Bugs fixed:
    - "exo-mount ignores HAL volume mount options 'sync' and 'noatime'"
      (LP: #238702)

exo (0.3.105-3) unstable; urgency=low

  * debian/control:
    - add an exo-utils-dbg package containing debug symbols for the exo-utils
      binaries.
  * debian/rules:
    - use variables for library name, soversion and version.

exo (0.3.105-2) unstable; urgency=low

  * debian/control:
    - add build-dep on quilt
  * debian/rules:
    - call {,un}patch targets when needed, include quilt rules.
  * debian/patches:
    - 01_mount-async added, don't mount all volumes with sync.

exo (0.3.105-1) unstable; urgency=low

  * New upstream release.
  * add a new libexo-common packages containing mount options file,
    documentation, helper files.
  * debian/rules:
    - simplify rules file.
  * debian/patches:
    - 01_fix-exo-csource-manpage dropped, upstream manpage fixed.
    - 02_fix-exo-open-manpage as well.
  * debian/control:
    - drop quilt build-dep.
 -- Lionel Le Folgoc <email address hidden> Sun, 08 Nov 2009 15:45:15 +0100

Changed in exo (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Lionel Le Folgoc (mrpouit) wrote :

Ok, I just tested, and I'll have to ship a mount.rc file in xubuntu-default-settings to pass utf8=true at least for vfat (maybe for iso9960 too...).

Changed in xubuntu-default-settings (Ubuntu):
assignee: nobody → Lionel Le Folgoc (mrpouit)
importance: Undecided → Low
status: Incomplete → Triaged
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package xubuntu-default-settings - 0.68

---------------
xubuntu-default-settings (0.68) lucid; urgency=low

  * Refreshed translations.
  * etc/xdg/xubuntu/xfce4:
    - helpers.rc: set default applications
    - mount.rc: set default mount options. LP: #238702
 -- Lionel Le Folgoc <email address hidden> Tue, 10 Nov 2009 21:55:45 +0100

Changed in xubuntu-default-settings (Ubuntu):
status: Triaged → Fix Released
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.