> On March 15, 2016, 6:57 a.m., Gleb Popov wrote: > > Can you, please, elaborate on pt. 2? What does not work exactly? > > > > >2) When compiling okularplugin, the existing definition of > > >okularpart_EXPORTS on the generated settings.cpp does not work with MinGW, > > >for reasons that I do not understand. Setting the definition on the whole > > >target, instead, works.
I don't have the exact message at hand, right now. However, it is linking okularplugin, that fails (several undefined reference errors, all for symbols from settings.cpp). Same problem with both MinGW 4.9.4 and 5.3.0. Again, I cannot see, why your existing workaround should not fix this, but it does not... - Thomas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127328/#review93529 ----------------------------------------------------------- On March 14, 2016, 12:17 a.m., Thomas Friedrichsmeier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127328/ > ----------------------------------------------------------- > > (Updated March 14, 2016, 12:17 a.m.) > > > Review request for Okular, Aleix Pol Gonzalez, Alex Richardson, and Gleb > Popov. > > > Repository: okular > > > Description > ------- > > Three parts to this patch: > 1) The plaform #ifdef in interfaces/viewerinterface.h is platform, not > compiler-specific. > 2) When compiling okularplugin, the existing definition of okularpart_EXPORTS > on the generated settings.cpp does not work with MinGW, for reasons that I do > not understand. Setting the definition on the whole target, instead, works. > 3) The okularpart lib will automatically be named libokularpart, and > subsequently not be found when trying to load it via KPluginLoader. Tell > cmake to drop the "lib"-prefix. > > See also: > - Discussion on list > https://mail.kde.org/pipermail/okular-devel/2016-March/022480.html > - A previous review request addressed issue 2 for MSVC > https://git.reviewboard.kde.org/r/125742/ > > > Diffs > ----- > > CMakeLists.txt e17bc25 > interfaces/viewerinterface.h e9e76a2 > mobile/components/CMakeLists.txt e09326a > > Diff: https://git.reviewboard.kde.org/r/127328/diff/ > > > Testing > ------- > > Builds on Windows with MinGW. Starts, and loads okularpart, successfully. > > > Thanks, > > Thomas Friedrichsmeier > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
