----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125741/#review87233 -----------------------------------------------------------
This is a workaround and will break when i decide to remove it again becuase the include is not needed there, are you sure there isn't a better way to fix it? - Albert Astals Cid On oct. 21, 2015, 6:07 p.m., Gleb Popov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125741/ > ----------------------------------------------------------- > > (Updated oct. 21, 2015, 6:07 p.m.) > > > Review request for Okular. > > > Repository: okular > > > Description > ------- > > okular/part.cpp includes interfaces/viewerinterface.h and its > OKULARCORE_EXPORT macro gets expanded into __declspec(dllimport). This makes > VS linker to drop ViewerInterface::~ViewerInterface() definition and to look > for it in KF5OkularCore.dll. But this include is never used anywhere in > okularcore, hence missing symbol. > > Including interfaces/viewerinterface.h in any of okularcore sources fixes the > problem. > > > Diffs > ----- > > core/document.cpp 49ce0650b4f970290490beb0db6d5c29c13311fa > > Diff: https://git.reviewboard.kde.org/r/125741/diff/ > > > Testing > ------- > > > Thanks, > > Gleb Popov > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
