Re: Review Request 114980: Do not explicity find or use Qt5WebKit(Widgets)
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114980/#review47942 --- Welp, since no-one seems to care, I'm gonna commit it. - Alex Merry On Jan. 12, 2014, 12:58 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114980/ --- (Updated Jan. 12, 2014, 12:58 p.m.) Review request for KDE Frameworks and Alex Merry. Repository: kdesignerplugin Description --- Do not explicity find or use Qt5WebKit(Widgets) The kdewebkit designer plugin does not directly use anything from Qt5WebKitWidgets, only KWebView from KF5WebKit. Qt5WebKitWidgets is found as a dependency of KF5WebKit. So we do not need to find it ourselves, or explicitly link against it. Diffs - CMakeLists.txt d13d071f935306df2d052b58a6a03ea92bba36f3 src/CMakeLists.txt 55ac819d4c2cbaac483cec9617402e18fbe22d2b Diff: https://git.reviewboard.kde.org/r/114980/diff/ Testing --- Still configures, builds and installs. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 114980: Do not explicity find or use Qt5WebKit(Widgets)
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114980/ --- (Updated Jan. 21, 2014, 10:42 p.m.) Status -- This change has been marked as submitted. Review request for KDE Frameworks and Alex Merry. Repository: kdesignerplugin Description --- Do not explicity find or use Qt5WebKit(Widgets) The kdewebkit designer plugin does not directly use anything from Qt5WebKitWidgets, only KWebView from KF5WebKit. Qt5WebKitWidgets is found as a dependency of KF5WebKit. So we do not need to find it ourselves, or explicitly link against it. Diffs - CMakeLists.txt d13d071f935306df2d052b58a6a03ea92bba36f3 src/CMakeLists.txt 55ac819d4c2cbaac483cec9617402e18fbe22d2b Diff: https://git.reviewboard.kde.org/r/114980/diff/ Testing --- Still configures, builds and installs. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Re: Review Request 114980: Do not explicity find or use Qt5WebKit(Widgets)
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114980/#review47943 --- This review has been submitted with commit 749336963bbcddea2e547020202e049937536275 by Alex Merry to branch master. - Commit Hook On Jan. 12, 2014, 12:58 p.m., Alex Merry wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114980/ --- (Updated Jan. 12, 2014, 12:58 p.m.) Review request for KDE Frameworks and Alex Merry. Repository: kdesignerplugin Description --- Do not explicity find or use Qt5WebKit(Widgets) The kdewebkit designer plugin does not directly use anything from Qt5WebKitWidgets, only KWebView from KF5WebKit. Qt5WebKitWidgets is found as a dependency of KF5WebKit. So we do not need to find it ourselves, or explicitly link against it. Diffs - CMakeLists.txt d13d071f935306df2d052b58a6a03ea92bba36f3 src/CMakeLists.txt 55ac819d4c2cbaac483cec9617402e18fbe22d2b Diff: https://git.reviewboard.kde.org/r/114980/diff/ Testing --- Still configures, builds and installs. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
Review Request 114980: Do not explicity find or use Qt5WebKit(Widgets)
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114980/ --- Review request for KDE Frameworks and Alex Merry. Repository: kdesignerplugin Description --- Do not explicity find or use Qt5WebKit(Widgets) The kdewebkit designer plugin does not directly use anything from Qt5WebKitWidgets, only KWebView from KF5WebKit. Qt5WebKitWidgets is found as a dependency of KF5WebKit. So we do not need to find it ourselves, or explicitly link against it. Diffs - CMakeLists.txt d13d071f935306df2d052b58a6a03ea92bba36f3 src/CMakeLists.txt 55ac819d4c2cbaac483cec9617402e18fbe22d2b Diff: https://git.reviewboard.kde.org/r/114980/diff/ Testing --- Still configures, builds and installs. Thanks, Alex Merry ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel