> On Sept. 22, 2014, 3:05 p.m., Eike Hein wrote: > > Hmm, overall approach seems fine I think :). Using the executable as search > > key might not always find the right package, but then using the desktop > > file name isn't any more reliable, and the user can use muon-discover to > > check the result anyway. > > > > There's a laundry list of minor things to do: > > > > - Separate files for PackageKitJob > > - Coding style fixes > > - Action needs "..." and should probably use the muon icon > > > > I'm happy to do all that stuff though. More important is making it > > build-time optional :). You have way more CMake fu than I do, mind doing > > that?
Will do, it's related to moving it into a separate file anyway. - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120318/#review67207 ----------------------------------------------------------- On Sept. 22, 2014, 2:54 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120318/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2014, 2:54 p.m.) > > > Review request for Plasma and Eike Hein. > > > Repository: plasma-desktop > > > Description > ------- > > I've been discussing with Eike having something like that for a while, I > finally managed to put something together that we could use in a future. > > It adds an entry on the menu that is called "Remove '<packagename>'" that > opens a software center. I set it to muon-discover for now, but this should > be iterated over. > > To do the lookup, it uses PackageKitQt. It probably should be an optional > dependency, but I want Eike to look into it first and decide how to do it > best. > > > Diffs > ----- > > CMakeLists.txt 7b794ff > applets/kicker/CMakeLists.txt 0688732 > applets/kicker/plugin/appsmodel.cpp b88d711 > > Diff: https://git.reviewboard.kde.org/r/120318/diff/ > > > Testing > ------- > > I uninstalled openarena, selfcompiled software cannot removed. > > The locking is not really noticeable on my system. We still probably want to > improve that but I don't think it would be terrible like this, only bad. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel