From: "Andreas Sturmlechner" <asturm@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: kde-apps/kdepim-runtime/files/, kde-apps/kdepim-runtime/
Date: Thu, 3 Oct 2019 17:42:56 +0000 (UTC) [thread overview]
Message-ID: <1570124543.f06daf5beb7b4f48d875ea818ba29d5ad385767c.asturm@gentoo> (raw)
commit: f06daf5beb7b4f48d875ea818ba29d5ad385767c
Author: Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
AuthorDate: Thu Oct 3 12:35:58 2019 +0000
Commit: Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
CommitDate: Thu Oct 3 17:42:23 2019 +0000
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f06daf5b
kde-apps/kdepim-runtime: Add another upstream race condition fix
Tested-by: Rutger van Bergen <rbergen <AT> xs4all.nl>
Closes: https://bugs.gentoo.org/691938
Package-Manager: Portage-2.3.76, Repoman-2.3.17
Signed-off-by: Andreas Sturmlechner <asturm <AT> gentoo.org>
...ime-19.04.3-ui_compactpage-race-condition.patch | 50 ++++++++++++++++++++++
.../kdepim-runtime/kdepim-runtime-19.04.3.ebuild | 5 ++-
.../kdepim-runtime/kdepim-runtime-19.08.1.ebuild | 2 +
3 files changed, 56 insertions(+), 1 deletion(-)
diff --git a/kde-apps/kdepim-runtime/files/kdepim-runtime-19.04.3-ui_compactpage-race-condition.patch b/kde-apps/kdepim-runtime/files/kdepim-runtime-19.04.3-ui_compactpage-race-condition.patch
new file mode 100644
index 00000000000..cfee2009495
--- /dev/null
+++ b/kde-apps/kdepim-runtime/files/kdepim-runtime-19.04.3-ui_compactpage-race-condition.patch
@@ -0,0 +1,50 @@
+From 8ec4db907c403b24685a961b48be5f8a288d9bd9 Mon Sep 17 00:00:00 2001
+From: Christophe Giboudeaux <christophe@krop.fr>
+Date: Wed, 2 Oct 2019 11:26:29 +0200
+Subject: Fix the dependencies on generated files differently.
+
+Some Gentoo users were still having build issues with the mbox resource.
+
+BUG: 410474
+FIXED-IN: 19.08.1
+---
+ resources/mbox/CMakeLists.txt | 17 +++++++++++++----
+ 1 file changed, 13 insertions(+), 4 deletions(-)
+
+diff --git a/resources/mbox/CMakeLists.txt b/resources/mbox/CMakeLists.txt
+index c52230f..ca8bc1e 100644
+--- a/resources/mbox/CMakeLists.txt
++++ b/resources/mbox/CMakeLists.txt
+@@ -14,10 +14,18 @@ set( mboxresource_SRCS
+ ${mboxresource_common_SRCS}
+ )
+
+-# Fix a race condition. mboxresource.cpp needs compactpage.h which includes ui_compactpage.h.
+-# Make sure the ui file is generated before building the akonadi_mbox_resource target.
+-set_source_files_properties(mboxresource.cpp PROPERTIES OBJECT_DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/ui_compactpage.h)
+-set_source_files_properties(mboxresource.cpp PROPERTIES OBJECT_DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/ui_lockfilepage.h)
++# mboxresource.cpp needs UI files generated for another target. We must be sure the files
++# were created before building the akonadi_mbox_resource target.
++add_custom_target(generated_headers
++ DEPENDS
++ ${CMAKE_CURRENT_BINARY_DIR}/ui_compactpage.h
++ ${CMAKE_CURRENT_BINARY_DIR}/ui_lockfilepage.h
++)
++set_source_files_properties(
++ ${CMAKE_CURRENT_BINARY_DIR}/ui_compactpage.h
++ ${CMAKE_CURRENT_BINARY_DIR}/ui_lockfilepage.h
++ PROPERTIES GENERATED TRUE
++)
+
+ ecm_qt_declare_logging_category(mboxresource_SRCS HEADER mboxresource_debug.h IDENTIFIER MBOXRESOURCE_LOG CATEGORY_NAME org.kde.pim.mboxresource)
+
+@@ -29,6 +37,7 @@ qt5_add_dbus_adaptor(mboxresource_SRCS
+ )
+
+ add_executable(akonadi_mbox_resource ${mboxresource_SRCS})
++add_dependencies(akonadi_mbox_resource generated_headers)
+
+ if( APPLE )
+ set_target_properties(akonadi_mbox_resource PROPERTIES MACOSX_BUNDLE_INFO_PLIST ${CMAKE_CURRENT_SOURCE_DIR}/../Info.plist.template)
+--
+cgit v1.1
diff --git a/kde-apps/kdepim-runtime/kdepim-runtime-19.04.3.ebuild b/kde-apps/kdepim-runtime/kdepim-runtime-19.04.3.ebuild
index a187fe09102..418ba920270 100644
--- a/kde-apps/kdepim-runtime/kdepim-runtime-19.04.3.ebuild
+++ b/kde-apps/kdepim-runtime/kdepim-runtime-19.04.3.ebuild
@@ -74,7 +74,10 @@ RDEPEND="${COMMON_DEPEND}
!kde-misc/akonadi-ews
"
-PATCHES=( "${FILESDIR}/${P}-ui_lockfilepage-race-condition.patch" )
+PATCHES=(
+ "${FILESDIR}/${P}-ui_lockfilepage-race-condition.patch"
+ "${FILESDIR}/${P}-ui_compactpage-race-condition.patch"
+)
RESTRICT+=" test"
diff --git a/kde-apps/kdepim-runtime/kdepim-runtime-19.08.1.ebuild b/kde-apps/kdepim-runtime/kdepim-runtime-19.08.1.ebuild
index 1afb7ce36c0..88e85dc3bb7 100644
--- a/kde-apps/kdepim-runtime/kdepim-runtime-19.08.1.ebuild
+++ b/kde-apps/kdepim-runtime/kdepim-runtime-19.08.1.ebuild
@@ -74,6 +74,8 @@ RDEPEND="${COMMON_DEPEND}
!kde-misc/akonadi-ews
"
+PATCHES=( "${FILESDIR}/${PN}-19.04.3-ui_compactpage-race-condition.patch" ) # bug #691938
+
RESTRICT+=" test"
src_prepare() {
next reply other threads:[~2019-10-03 17:43 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-03 17:42 Andreas Sturmlechner [this message]
-- strict thread matches above, loose matches on Subject: below --
2020-08-01 22:57 [gentoo-commits] repo/gentoo:master commit in: kde-apps/kdepim-runtime/files/, kde-apps/kdepim-runtime/ Andreas Sturmlechner
2019-08-14 8:47 Andreas Sturmlechner
2017-04-22 14:32 Andreas Sturmlechner
2017-03-25 1:13 Michael Palimaka
2016-12-19 20:08 Johannes Huber
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1570124543.f06daf5beb7b4f48d875ea818ba29d5ad385767c.asturm@gentoo \
--to=asturm@gentoo.org \
--cc=gentoo-commits@lists.gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox