----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127948/#review95559 -----------------------------------------------------------
+1 from me. Before pushing, can you please add a unit test for this behaviour? Otherwise looks good to go. - Matthew Dawson On May 17, 2016, 3:57 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127948/ > ----------------------------------------------------------- > > (Updated May 17, 2016, 3:57 p.m.) > > > Review request for KDE Frameworks and Matthew Dawson. > > > Repository: kconfig > > > Description > ------- > > Previously, if a .desktop file had a TryExec and a X-KDE-AuthorizeAction > entry we would skip the KAuthorized check. This is clearly wrong. > > > Diffs > ----- > > src/core/kdesktopfile.cpp eda7c292da9aa2bc97f6066e90034767134c2e07 > > Diff: https://git.reviewboard.kde.org/r/127948/diff/ > > > Testing > ------- > > Disabled shell_access in kde5rc. > > With I was unable to open konsole from kicker. > Opening dolphin still worked as normal. > > > Thanks, > > David Edmundson > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel