> On Okt. 29, 2014, 12:02 nachm., Helio Castro wrote: > > Ok, way better than the previous one :-)
Thanks for your approval. Now who is going to commit my patch? I don't have write access to the repository, as far as I know. - Frank ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120839/#review69425 ----------------------------------------------------------- On Okt. 28, 2014, 11:37 nachm., Frank Schütte wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120839/ > ----------------------------------------------------------- > > (Updated Okt. 28, 2014, 11:37 nachm.) > > > 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 > >
