----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120839/#review69425 -----------------------------------------------------------
Ship it! Ok, way better than the previous one :-) - Helio Castro On Oct. 28, 2014, 11:37 p.m., Frank Schütte wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120839/ > ----------------------------------------------------------- > > (Updated Oct. 28, 2014, 11:37 p.m.) > > > Review request for kdelibs, Solid, Christoph Feck, and Helio Castro. > > > Bugs: 243046 and 261376 > http://bugs.kde.org/show_bug.cgi?id=243046 > http://bugs.kde.org/show_bug.cgi?id=261376 > > > Repository: kde-runtime > > > Description > ------- > > Hi Christoph, > solid-device-automounter fails to automount unknown devices, even if it is > checked, > because in kded/DeviceAutomounter.cpp it fails to evaluate > shouldAutomountDevice. > So this patch corrects the logic to at first evaluate shouldAutomountDevice. > Inside this evaluation automountUnknownDevices is evaluated correctly. > Please review this patch. Christoph Feck urged me to submit this patch. > It took me quite a while to figure it out, though. > Bye, > Frank > > > Diffs > ----- > > solid-device-automounter/kded/DeviceAutomounter.cpp 14b4e87 > solid-device-automounter/lib/AutomounterSettings.cpp 2b3e6be > > Diff: https://git.reviewboard.kde.org/r/120839/diff/ > > > Testing > ------- > > > Thanks, > > Frank Schütte > >
_______________________________________________ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel