----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106378/#review18899 -----------------------------------------------------------
Ship it! >From where I stand, I see HAL as the official backend for freeBSD, no modern >linux distro (or any distro that will update their libsolid version) is >shipping with HAL. So, if with this patch freeBSD support is better, please go ahead! - Alex Fiestas On Sept. 8, 2012, 12:26 a.m., Alberto Villa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106378/ > ----------------------------------------------------------- > > (Updated Sept. 8, 2012, 12:26 a.m.) > > > Review request for Solid. > > > Description > ------- > > Current hack to check for removable property in StorageAccess devices goes up > only one level to search for the StorageDrive device. This works fine with > UDev, but not with HAL, which can have (at least on FreeBSD, where I'm > testing it) a StorageVolume device in the middle. Going up the whole tree of > the Block device ensures that we eventually get to the StorageDrive one to > fetch the correct removable property. > > While here, I'd like to ask why only removable is set (in a not so convincing > way, as a removable device can be non-hotpluggable as far as I know) instead > of both removable and hotpluggable, as done few lines above. > > If approved, I'd like to commit this also to 4.9. > > > Diffs > ----- > > plasma/generic/dataengines/soliddevice/soliddeviceengine.cpp 86f123c > > Diff: http://git.reviewboard.kde.org/r/106378/diff/ > > > Testing > ------- > > Tested successfully on FreeBSD 10-CURRENT r239665 with KDE SC 4.9.0: my USB > flash drives now appear in the device notifier plasmoid (and a console.log() > in the plasmoid itself confirms that the device now has the Removable > property). > > I also double-checked with a UDev-backed `solid-hardware list details` log > that the logic correctly applies to UDev. > > > Thanks, > > Alberto Villa > >
_______________________________________________ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel