[Xubuntu, Studio?, Lubuntu?] Disable automount

Bug #719338 reported by Lionel Le Folgoc
26
This bug affects 4 people
Affects Status Importance Assigned to Milestone
ubiquity (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: ubiquity

Hi,

Ubiquity integrates with thunar-volman in Xfce 4.4 and 4.6 (Bug #107259). This was rather straightforward, because thunar-volman used to store its configuration in a simple rc file (read file, update entries if they exist, add them if they are missing, and restore them after the installation).

However, in Xfce 4.8, thunar-volman moved to xfconf, which means that its conf is now in xml (yay for parsing!), and managed by xfconfd (so any manual edition of the conf will be overwritten by xfconfd). Unfortunately, there are no python bindings released/packaged for xfconf, so I'm not sure of the best way to do it...

My skills in python are not that great, so I used a test script (instead of patching ubiquity directly), so I could test more easily.
What it does:
* read the previous stored user config, if it exists (if the user didn't change the settings, it won't).
* launch xfconf-query with he correct options to disable automounting.
* launch xfconf-query to restore the previous settings (either reset to the values set by the user, or reset to the system values if the stored config didn't exist).
It uses minidom to parse the config, and then spawns xfconf-query several times using os.system (yeah, this is not that nice -- it's probably not very robust either, that's why I didn't provide a patch directly also).

Feel free to improve it, or if you have any better way of doing that it'd be great.
I can also do some testing if needed.
Thanks!

Tags: iso-testing

Related branches

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

I also attach another xml config, to show more possible things (type="empty" means that the default system config is used, and that the user didn't change this property, so it doesn't need to be restored afterwards, only reset).

Revision history for this message
Charlie Kravetz (cjkgeek) wrote :

Thank you for submitting this bug report. I am confirming it, based on the comments made.

Changed in ubiquity (Ubuntu):
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Dear xubuntu-devs, would you like to change your default livecd settings to stop automounting or would you like ubiquity to do subprocess.call to flip that key off:

xfconf-query -c thunar-volman -p /automount-drives/enabled -s false

summary: - [Xubuntu] Disable automount
+ [Xubuntu, Studio?, Lubuntu?] Disable automount
Changed in ubiquity (Ubuntu):
status: Confirmed → Triaged
Revision history for this message
Colin Watson (cjwatson) wrote :

I strongly recommend that we change ubiquity, regardless of what the desktop maintainers do. Even if all in-archive flavours change their configuration (which they may or may not want to do), we don't want to end up accumulating support debt from derivatives who don't realise that they need to do this to make the installer work. ubiquity already has plenty of different ways in which it attempts to disable automounting; one more won't hurt.

Revision history for this message
Ubuntu QA Website (ubuntuqa) wrote :

This bug has been reported on the Ubuntu ISO testing tracker.

A list of all reports related to this bug can be found here:
http://iso.qa.ubuntu.com/qatracker/reports/bugs/719338

tags: added: iso-testing
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubiquity - 2.11.31

---------------
ubiquity (2.11.31) quantal; urgency=low

  [ Colin Watson ]
  * Buffer reads from debconf-copydb. Python 3 defaults to unbuffered reads
    from byte streams, which is much slower.
  * Port oem-config-remove-gtk to Python 3, now that
    python3-aptdaemon.gtk3widgets exists; the final piece!
  * Remove Python 2 support. We aren't going back now.
  * Remove the rest of the long-dead PS3 port.

  [ Dmitrijs Ledkovs ]
  * Allow going from the encryption key setup page, to the disk space
    allocation page. (LP: #1046323)
  * Correctly apply styles to the title & progress sections and not the
    centre piece. This means themes are no longer required to provide non
    standard @dark_[fb]g_color colors, instead 'menubar' Gtk CSS style is
    used. This also fixes long standing a11y issues in HighContrast themes
    & requirements for non-standard color definitions. (LP: #744283)
  * Make lables in the segmented bar use the same colours as normal labels.
  * Changing the "Add/Remove/Change Partiontion" buttons to mixed
    symbolic/text buttons should make the screen comply with Ubiquity
    Design spec ( http://goo.gl/Kokw5 ) and address confusion about the
    updated screen. (LP: #1045799)
  * Fix the disappearance of crypto password field (LP: #1045716)
  * Fix misallignment of crypto password fields (LP: #1045712)
  * Make setup security key page go back to ask page (LP: #1045698)
  * Remove powerpc/ps3 low-memory profile, as it doesn't do anything any
    more. gnome-session-remove command is no longer present, and the init
    processes it tried to stop no longer exist under those names. We do
    want a low memory profile, which ubiquity-only mode now.
  * ubiquity-wrapper: Add support for udisks2 inhibit, drop devkit-disks
    inhibit. (LP: #719338)
  * Prevent progress label to expand & shrink the window (LP: #1046241)
  * Automatic update of included source packages: base-installer
    1.122ubuntu9.

  [ Dylan McCall ]
  * Fixed slideshow_get_available_locale missing new locales in
    extra_slides_dir. (LP: #1046511)
 -- Dmitrijs Ledkovs <email address hidden> Sat, 08 Sep 2012 00:47:54 +0100

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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