----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130022/#review102895 -----------------------------------------------------------
Ship it! There was a test that needed an extra if. I've added it an pushed on your behalf. - Albert Vaca Cintora On March 17, 2017, 5:58 p.m., Oliver Sander wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/130022/ > ----------------------------------------------------------- > > (Updated March 17, 2017, 5:58 p.m.) > > > Review request for 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. > > This is continuation of https://git.reviewboard.kde.org/r/126378/ , where I > am not allowed to post patches. > > > Diffs > ----- > > CMakeLists.txt 2f75a9f1 > shell/CMakeLists.txt 59a8d76d > shell/shell.h 20e7c053 > shell/shell.cpp 448b9ff7 > > Diff: https://git.reviewboard.kde.org/r/130022/diff/ > > > Testing > ------- > > None, really, because I don't have a windows machine. > > > Thanks, > > Oliver Sander > >