[SCM] marble packaging branch, kubuntu_wily_archive, updated. debian/4.14.2-1-42-g400d103

Philip Muškovac yofel-guest at moszumanska.debian.org
Mon Aug 17 21:07:04 UTC 2015


Gitweb-URL: http://git.debian.org/?p=pkg-kde/applications/marble.git;a=commitdiff;h=400d103

The following commit has been merged in the kubuntu_wily_archive branch:
commit 400d1030d5ed8b96b26595d6c464a836cdba916b
Author: Philip Muškovac <yofel at gmx.net>
Date:   Mon Aug 17 23:07:01 2015 +0200

    Add upstream_fix-vtable-linking.diff to fix vtable linking
---
 debian/changelog                                |  3 ++
 debian/patches/series                           |  1 +
 debian/patches/upstream_fix-vtable-linking.diff | 45 +++++++++++++++++++++++++
 3 files changed, 49 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 5b48a80..cbb248a 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -9,6 +9,9 @@ marble (4:15.07.90-0ubuntu1) UNRELEASED; urgency=medium
   * Port to frameworks 5
   * new upstream beta release
 
+  [ Philip Muškovac ]
+  * Add upstream_fix-vtable-linking.diff to fix vtable linking
+
  -- Jonathan Riddell <jriddell at ubuntu.com>  Thu, 13 Aug 2015 11:44:25 +0100
 
 marble (4:15.04.2-0ubuntu2) wily; urgency=medium
diff --git a/debian/patches/series b/debian/patches/series
index 8aa31ed..9dbb854 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,3 +1,4 @@
 #fix_qmake_fallback.diff
 #do_not_install_private_headers
 #kubuntu_disable-MarbleRunnerManagerTest.diff
+upstream_fix-vtable-linking.diff
diff --git a/debian/patches/upstream_fix-vtable-linking.diff b/debian/patches/upstream_fix-vtable-linking.diff
new file mode 100644
index 0000000..f4491c8
--- /dev/null
+++ b/debian/patches/upstream_fix-vtable-linking.diff
@@ -0,0 +1,45 @@
+Description: add missing main.moc include for plasmarunner and use cmake settings
+ why exactly the build works for anyone without the moc is beyond me, the
+ include however is supposedly necessary as documented at [1]. otherwise
+ the factory (object created by the macro) metaobject code is not present
+ and the factory linking fails on the missing vtable. this is reproducible
+ on cmake 3.2.2, gcc5 and frameworks 5.13.
+ .
+ additionally include the KDECMakeSettings from ECM as to enable
+ current_binary_directory includes by default (which is necessary to
+ find main.moc)
+ .
+ [1] https://techbase.kde.org/Development/Tutorials/Plasma4/AbstractRunner#The_Plugin_Declaration_Macro
+Author: Harald Sitter <sitter at kde.org>
+Forwarded: https://git.reviewboard.kde.org/r/124787/
+Origin: vendor
+diff --git a/src/plasmarunner/CMakeLists.txt b/src/plasmarunner/CMakeLists.txt
+index 32591ef6bfc7b41ca644f5ec34fbdb97225f5749..7a68d097cb36a86dd83d66c43f0cd32d89a3476d 100644
+--- a/src/plasmarunner/CMakeLists.txt
++++ b/src/plasmarunner/CMakeLists.txt
+@@ -5,6 +5,7 @@ find_package(ECM QUIET)
+ set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} ${ECM_MODULE_PATH})
+ if(ECM_FOUND)
+   include(KDEInstallDirs)
++  include(KDECMakeSettings)
+ endif()
+ include(FeatureSummary)
+ macro_optional_find_package(KF5 QUIET COMPONENTS Config Runner Service I18n)
+@@ -12,7 +13,7 @@ macro_optional_find_package(KF5 QUIET COMPONENTS Config Runner Service I18n)
+ if (NOT KF5_FOUND)       
+   return()
+ endif()
+-
++  
+ set(plasmaRunner_SRCS
+   main.cpp
+   plasmarunner.cpp
+diff --git a/src/plasmarunner/main.cpp b/src/plasmarunner/main.cpp
+index 43c30bb2dcb3d078e6f214aaab23e2688399354f..3f60f9fab149ffc6a862f03af770b077b991e894 100644
+--- a/src/plasmarunner/main.cpp
++++ b/src/plasmarunner/main.cpp
+@@ -17,3 +17,4 @@
+ 
+ K_EXPORT_PLASMA_RUNNER(marble, Marble::PlasmaRunner)
+ 
++#include "main.moc"

-- 
marble packaging



More information about the pkg-kde-commits mailing list