----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126378/#review89610 -----------------------------------------------------------
shell/shell.cpp (line 48) <https://git.reviewboard.kde.org/r/126378/#comment61352> `#if HAVE_KACTIVITIES` would be nicer and then KActivities could just be turned into an optional dependency which wouldn't be found on Windows? I prefer having ifdefs based on available features rather than hardcoded on the OS - Alex Richardson On Dec. 16, 2015, 1:16 a.m., Patrick Spendrin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126378/ > ----------------------------------------------------------- > > (Updated Dec. 16, 2015, 1:16 a.m.) > > > Review request for kdewin and Okular. > > > Repository: okular > > > Description > ------- > > KActivities is not used on Windows, so a hard dependency on it > only brings unneeded code. > This patch removes it completely under Windows. > > > Diffs > ----- > > CMakeLists.txt 72ff3e8754a9005dd09cf49e83d67b0a4f9258db > shell/CMakeLists.txt 7a6af745464fe7bb1d6702cc02787e7e6d4c101e > shell/shell.cpp e69ecbb5e1bde36dbd00e5ff05cdad4e3d2a5f8d > > Diff: https://git.reviewboard.kde.org/r/126378/diff/ > > > Testing > ------- > > Windows. > > > Thanks, > > Patrick Spendrin > >
_______________________________________________ Kde-windows mailing list Kde-windows@kde.org https://mail.kde.org/mailman/listinfo/kde-windows
