Mediamanager in kde 3.5.5 does not honour automount setting

Bug #69561 reported by Marco Antonucci
22
Affects Status Importance Assigned to Milestone
kdebase (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Removable devices gets always mounted regardless of what is selected in "properties->mounting->automatically mount" (default: unchecked). Tested on both Dapper (with kde 3.5.5) and Edgy. Until kde 3.5.4 default behaviour was that media devices were shown on the desktop without actually mounting anything; but right-clicking on devices and checking "properties->mounting->automatically mount" worked as expected.

Revision history for this message
Marco Antonucci (marco-antonucci) wrote :

I think the cause of this bug resides in patch "kubuntu_73_nomediaioslave_mounthelper.diff". I'm not a c++ developer, but, as i understand, the function "automountRemovables()" inserted in medianotifier.cpp mounts removable devices as soon as they are found overriding the settings in mediamanager.

Revision history for this message
Martin Lubich (mlubich) wrote :

I have to confirm this behaviour.
Furthermore not only the automount property gets ignored but also the mount point as well as the mount options selectable through the mount properties dialog.
On Dapper it worked until an upgrade to hal from 0.5.7-1ubuntu18.2 to 0.5.7-1ubuntu18.2.
On Edgy it didn't work right from the beginning, and no update (including KDE 3.5.6) did correct this.

Revision history for this message
Yuriy Kozlov (yuriy-kozlov) wrote :

Confirmed according to comments and duplicate.

Changed in kdebase:
importance: Undecided → Medium
status: Unconfirmed → Confirmed
Revision history for this message
lexual (lexhider) wrote :

http://bugs.kde.org/show_bug.cgi?id=141502
There is a bug in kde bugzilla regarding this.
I suspect this is kubuntu specific.
If this is the case could someone please close the bug or let me know and I will do it.

Revision history for this message
Yuriy Kozlov (yuriy-kozlov) wrote :

If it's in the patch that Marco mentioned, then it is indeed kubuntu specific.

Revision history for this message
Konrad Materka (kmaterka) wrote :

The same problem in Edgy in KDE 3.5.6 from:
deb http://kubuntu.org/packages/kde-356 edgy main
Kubuntu's backport.
Hope it will be fixed soon :) Problem with Hal hadn't been fixed for so long... :(

Revision history for this message
Marc Schiffbauer (mschiff) wrote :

This nasty bug is still in feisty :-(

Revision history for this message
Marc Schiffbauer (mschiff) wrote :

The 3.5.7 packages still have this bug.
There is no such problem with the OpenSUSE packages.

Could the buggy patch added to kubuntu packages simply be dropped?

Revision history for this message
Marco Antonucci (marco-antonucci) wrote :

It's since KDE 3.5.5 on Dapper that i rebuild kdebase packages to fix this "feature".
Now i'm still on Edgy, so i can't test 3.5.7. With KDE 3.5.6 i've commented out lines 155 and 387 in patch kubuntu_73_nomediaioslave_mounthelper.diff

I've also dropped patch kubuntu_83_media_desktop.diff

Revision history for this message
Marc Schiffbauer (mschiff) wrote :

Sounds good. Could you be a bit more precise please? I would fix this then a build a new kdebase package.

Are you sure these patches are part of the kdebase package? I just apt-get_source'd kdebase but cannot find patches named like the ones you mentioned.

How did the lines you commented out look like?

In the current kubuntu kdebase packages there are these patches:

s -1R debian/patches/
debian/patches/:
10_kdmrc_defaults.diff
11_genkdmconf.diff
12_genkdm_make.diff
13_kdm_sessions_fixes.diff
14_kdm_no_custom.diff
15_cups_docroot.diff
16_kdm_X_path.diff
17_logitechmouse.diff
18_default_fonts.diff
19_kcontrol_only_in_kde.diff
20_applications_menu_move.diff
23_startkde_kpersonalizer.diff
24_knotify_arts.diff
25_khelp_htdig.diff
26_face_load.diff
27_kfreebsd_sysguard.diff
28_kdebase_rubberband.diff
34_japanese_konsole_fix.diff
38_furry_frosted_animals_support.diff
39_be_better_at_honouring_user_kdm_theming.diff
40_kdm_override_docs.diff
41_nspluginviewer_64bit_fix.diff
43_kdeprintfax_addquotes.diff
98_buildprep.diff
common

debian/patches/common:
03_libtool_update.diff
04_am_maintainer_mode.diff
05_pedantic-errors.diff
06_disable_no_undefined.diff
README
update.sh

Revision history for this message
Marco Antonucci (marco-antonucci) wrote :

I've manually downloaded kdebase_3.5.7-0ubuntu1~feisty1.dsc, kdebase_3.5.7-0ubuntu1~feisty1.diff.gz and kdebase_3.5.7.orig.tar.gz. Applied dpkg-source -x and looked into generated kdebase-3.5.7 directory: i can confirm the patches i've mentioned are present in /debian/patches.

The lines i've commented are 155 from:

+ ensureMediumMounted(medium.url().fileName(), ok);

to:

+ //ensureMediumMounted(medium.url().fileName(), ok);

and 387 from:

+ automountRemovables();

to:

+// automountRemovables();

Revision history for this message
Marc Schiffbauer (mschiff) wrote :

Seems to be fixed in gutsy. Works for me now after upgrading.

Revision history for this message
Marco Antonucci (marco-antonucci) wrote :

Fixed also in feisty with kde 3.5.8 upgrade.

Changed in kdebase:
status: Confirmed → 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.