> On July 13, 2015, 11:18 a.m., Laurent Montel wrote: > > I didn't test compile so I hope that it's ok. > > Please commit in master > > Pali Rohár wrote: > Compilation test is not enough, specially for plugins. On Linux when > building shared libraries (=also plugins) gcc by default allows you to create > library with unreferenced symbols. It means that gcc/ld does not check if > every symbol (function, variable) is found in some linked library. > > So doing runtime test for any change which remove library dependency from > plugins is *REQUIRED*.
I'll check again. Thanks. - R.Harish ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124332/#review82448 ----------------------------------------------------------- On July 12, 2015, 7:44 p.m., R.Harish Navnit wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124332/ > ----------------------------------------------------------- > > (Updated July 12, 2015, 7:44 p.m.) > > > Review request for Kopete, Laurent Montel and Pali Rohár. > > > Repository: kopete > > > Description > ------- > > Some plugins have been ported and compile without KDE3/Qt3 support libraries. > > > Diffs > ----- > > plugins/alias/CMakeLists.txt ddfe3c24df9b3668e1b3194e626c22b2f6bb55b6 > plugins/autoreplace/CMakeLists.txt a7a2e90402a503d5ced1166e55bcb82189f1f8ff > plugins/nowlistening/CMakeLists.txt > 4444d910afc14c00c4bac80b934753aadb70561d > plugins/texteffect/CMakeLists.txt 892e732004d371c9276040bd35d52d8f06e789e6 > > Diff: https://git.reviewboard.kde.org/r/124332/diff/ > > > Testing > ------- > > Build succeeded. > > > Thanks, > > R.Harish Navnit > >
_______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel
