Bug 751879
Summary: | solid hardware does not detect NFS drives, related to halectomy patch? | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Rudd-O DragonFear <rudd-o> |
Component: | kdelibs | Assignee: | Rex Dieter <rdieter> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 16 | CC: | fedora, jreznik, kevin, ltinkl, rdieter, rnovacek, ry, smparrish, than |
Target Milestone: | --- | Keywords: | Reopened |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | kdelibs-4.7.3-5.fc16 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2011-12-19 14:46:04 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Rudd-O DragonFear
2011-11-07 21:40:01 UTC
More info; https://bugs.kde.org/show_bug.cgi?id=269301 > Someone has patched the solid libraries to skip processing fstab, apparently.
Yes, that's exactly what's going on.
Lukáš, can we drop that patch which drops the fstab backend? Why do we have that patch in the first place? The fstab backend is clearly good enough for upstream and for most if not all other distributions…
Yes for the love of god, that bug has lingered in the kde bug database without anyone being able to figure out what's wrong, precisely because we patch it. The fstab backend is ESSENTIAL for many KDE services, including NETWORK removable device detection for the indexer and various multimedia applications such as Amarok. The patch BROKE several functionalities of these applications (such as, for example, proper indexing of media files in network-attached servers). It also BROKE KDE's auto detection of network mounts which allows dolphin and konqueror to display network-friendly behavior on mounts that could go away at any point. kdelibs-4.6.80-halectomy.patch:- grep fstab * kdelibs-4.6.80-halectomy.patch:- backends/hal/halfstabhandling.cpp kdelibs-4.6.80-halectomy.patch:- message(STATUS "Building Solid fstab backend." ) kdelibs-4.6.80-halectomy.patch:- backends/fstab/fstabmanager.cpp kdelibs-4.6.80-halectomy.patch:- backends/fstab/fstabdevice.cpp kdelibs-4.6.80-halectomy.patch:- backends/fstab/fstabnetworkshare.cpp kdelibs-4.6.80-halectomy.patch:- backends/fstab/fstabstorageaccess.cpp kdelibs-4.6.80-halectomy.patch:- backends/fstab/fstabhandling.cpp kdelibs-4.6.80-halectomy.patch:- backends/fstab/fstabwatcher.cpp kdelibs-4.6.80-halectomy.patch:-#include "backends/fstab/fstabmanager.h" This is the problematic patch. Guys, HALectomy does not imply removing fstab support. I think the solution is to simply compile with --without-hal or the equivalent CMake option that kdelibs provides. It HAS to be a compile-time option because there are platforms out there, such as solaris, that do NOT sport HAL. Also, the patch removes UPower. WHY? It's not a halectomy, it's a lobotomy! If the patch is going to do three or four things, better roll three or four separate patches. Uhm, no, it doesn't remove UPower, only the comma at the end of the UPower line because it removes the fstab line below. :-) Still, I don't think removing the fstab backend is a good idea, upstream ships it for a reason. from irc [16:32] <rudd-o> Kevin_Kofler: the fstab backend is the fallback in solid. it should be included *regardless* [16:32] <rudd-o> if ltinkl considers it "not good enough" for some reason, he should (1) have made TWO different patches, one to remove hal and other to remove fstab (2) made and upstreamed fixes to the fstab backend [16:32] <rudd-o> that's what SHOULD have happened [16:32] <Jucato> rudd-o: thanks :) [16:33] <rudd-o> atm I just want the patch to be removed / altered to exclusively neuter HAL, but leave the fstab backed in [16:33] <ltinkl> rudd-o: feel free to assign that bug to me, I will have a look at it tomorrow [16:34] <rudd-o> k, lukas [16:34] <rudd-o> I will [16:34] <ltinkl> no time / too sleepy now [16:34] <rudd-o> it is already assigned to you re-assigning to kdelibs. Fwiw, looks like some history of this patch, and why the fstab backend is ommited comes from upstream bug http://bugs.kde.org/261359 "using device notification to mount usbpen results in 100% cpu usage" But, also seems that fixing http://bugs.kde.org/show_bug.cgi?id=261541 may also have fixed the deadlock. so, let's drop it, and give it a whirl. Should we really drop the whole patch, as opposed to just the part which removes the fstab backend? I think removing the HAL backend entirely still makes sense… OK, that's the new plan (confirmed with ltinkl on irc). will revive halectomy patch to only omit hal, nothing more. YEAH!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! kdelibs-4.6.5-8.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/kdelibs-4.6.5-8.fc15 kdelibs-4.6.5-8.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/kdelibs-4.6.5-8.fc14 kdelibs-4.6.5-8.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report. kdelibs-4.6.5-8.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. This is NOT FIXED in Fedora 16: ~@karen.dragonfear α: rpm -q kdelibs kdelibs-4.7.3-2.fc16.x86_64 I see nothing in admin.fedoraproject.org indicating that an update for Fedora 16 was pushed, is pending, or is even slated for rebuild. ~@karen.dragonfear α: yum list kdelibs --enablerepo='kde*' Loaded plugins: auto-update-debuginfo, downloadonly, fastestmirror, fs-snapshot, presto, priorities, refresh-packagekit, remove-with-leaves, rpm- : warm-cache, show-leaves, upgrade-helper, verify, versionlock Loading mirror speeds from cached hostfile * fedora: mirrors.kernel.org * fedora-debuginfo: mirrors.kernel.org * kde: kdeforge.unl.edu * kde-testing: kdeforge.unl.edu * kde-unstable: kdeforge.unl.edu * rpmfusion-free: mirror.web-ster.com * rpmfusion-free-debuginfo: mirror.web-ster.com * rpmfusion-free-updates: mirror.web-ster.com * rpmfusion-free-updates-debuginfo: mirror.web-ster.com * rpmfusion-nonfree: mirror.web-ster.com * rpmfusion-nonfree-debuginfo: mirror.web-ster.com * rpmfusion-nonfree-updates: mirror.web-ster.com * rpmfusion-nonfree-updates-debuginfo: mirror.web-ster.com * updates: mirrors.kernel.org * updates-debuginfo: mirrors.kernel.org kde-unstable | 1.9 kB 00:00 kde-unstable/primary_db | 21 kB 00:00 Installed Packages kdelibs.x86_64 6:4.7.3-2.fc16 @kde-testing Available Packages kdelibs.i686 6:4.7.3-2.fc16 updates It's fixed in kdelibs >= 4.7.3-4.fc16, but we haven't filed an update for it yet. (Rex, do we want to file -4 or -5? -5 has the restored patch to remove the HAL backend (but not the fstab one) entirely, but it looks like you and Lukáš more or less agreed on IRC to just drop that patch, so maybe just go with -4?) Yes. I found http://koji.fedoraproject.org/koji/buildinfo?buildID=274525 indicating the build. I downloaded -5 and I am testing it. -------------------------------- ~/tmp α: solid-hardware query 'IS StorageAccess' udi = '/org/freedesktop/UDisks/devices/sda1' udi = '/org/kde/fstab/10.254.102.1:/export/shared/Software' udi = '/org/kde/fstab/10.254.102.1:/export/shared/Entertainment/TV' udi = '/org/kde/fstab/10.254.102.1:/export/shared/Entertainment/Miro' udi = '/org/kde/fstab/10.254.102.1:/export/shared/' udi = '/org/kde/fstab/10.254.102.1:/export/shared/Knowledge' udi = '/org/kde/fstab/10.254.102.1:/export/shared/Entertainment/Music' udi = '/org/kde/fstab/10.254.102.1:/export/shared/Entertainment/Movies' udi = '/org/kde/fstab/10.254.102.1:/export/shared/Entertainment' udi = '/org/kde/fstab/10.254.102.1:/export/shared/Incoming' udi = '/org/kde/fstab/10.254.102.1:/export/shared/Entertainment/Jokes' -------------------------------- Mmmmmgoood. You http://i.imgur.com/TbiNz.jpg the package :-) Thanks. No update has been pushed (in fact, not even filed) for Fedora 16, so closing the bug as ERRATA doesn't make sense. Also, you're not Rex Dieter, so why have you cleared the needinfo for him? I still need him to answer my question (or to just file the version he wants to push as an update himself). Sorry, I fail at bugzilla :-) kdelibs-4.7.3-5.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/kdelibs-4.7.3-5.fc16 kdelibs-4.7.3-5.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report. closing now that kdelibs-4.7.3-5.fc16 is out in the wild I confirm, it works. |