[libreoffice/f16] Resolves: rhbz#747356 let Qt call XInitThreads

Caolan McNamara caolanm at fedoraproject.org
Fri Oct 21 11:53:15 UTC 2011


commit cfb00fd1a0365cc44f32ad9671bf3235c414bec3
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Fri Oct 21 12:53:13 2011 +0100

    Resolves: rhbz#747356 let Qt call XInitThreads

 ...-XInitThreads-so-that-it-knows-it-s-been-.patch |   38 ++++++++++++++++++++
 libreoffice.spec                                   |    7 +++-
 2 files changed, 44 insertions(+), 1 deletions(-)
---
diff --git a/0001-let-Qt-call-XInitThreads-so-that-it-knows-it-s-been-.patch b/0001-let-Qt-call-XInitThreads-so-that-it-knows-it-s-been-.patch
new file mode 100644
index 0000000..f7f0229
--- /dev/null
+++ b/0001-let-Qt-call-XInitThreads-so-that-it-knows-it-s-been-.patch
@@ -0,0 +1,38 @@
+From 6163c62a52794725188faf82ee12852f4b2cd023 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Lubo=C5=A1=20Lu=C5=88=C3=A1k?= <l.lunak at suse.cz>
+Date: Thu, 20 Oct 2011 16:41:50 +0200
+Subject: [PATCH] let Qt call XInitThreads(), so that it knows it's been
+ called (fdo#40298)
+
+Otherwise QPixmap complains when used outside of the main Qt thread
+and resets itself to null pixmap, eventually leading to crashes.
+---
+ vcl/unx/kde4/main.cxx |   11 +++++++++++
+ 1 files changed, 11 insertions(+), 0 deletions(-)
+
+diff --git a/vcl/unx/kde4/main.cxx b/vcl/unx/kde4/main.cxx
+index 9e0a361..482a212 100644
+--- a/vcl/unx/kde4/main.cxx
++++ b/vcl/unx/kde4/main.cxx
+@@ -56,7 +56,18 @@ extern "C" {
+            established, so protect X against itself
+         */
+         if( ! ( pNoXInitThreads && *pNoXInitThreads ) )
++        {
++#if QT_VERSION >= 0x040800
++            // let Qt call XInitThreads(), so that also Qt knows it's been used
++            // (otherwise QPixmap may warn about threads not being initialized)
++            QApplication::setAttribute( Qt::AA_X11InitThreads );
++#else
+             XInitThreads();
++            // just in case somebody builds with old version and then upgrades Qt,
++            // otherwise this is a no-op
++            QApplication::setAttribute( static_cast< ApplicationAttribute >( 10 ));
++#endif
++        }
+ 
+ #if QT_VERSION < 0x050000
+         // Qt 4.x support needs >= 4.1.0
+-- 
+1.7.6.4
+
diff --git a/libreoffice.spec b/libreoffice.spec
index d91bb36..ee98e6b 100644
--- a/libreoffice.spec
+++ b/libreoffice.spec
@@ -27,7 +27,7 @@ Summary:        Free Software Productivity Suite
 Name:           libreoffice
 Epoch:          1
 Version:        3.4.3.2
-Release:        12%{?dist}
+Release:        13%{?dist}
 License:        LGPLv3 and LGPLv2+ and BSD and (MPLv1.1 or GPLv2 or LGPLv2 or Netscape) and (CDDL or GPLv2) and Public Domain
 Group:          Applications/Productivity
 URL:            http://www.documentfoundation.org/develop
@@ -127,6 +127,7 @@ Patch32: fdo40856.bn.discard.patch
 Patch33: 0001-Fix-for-fdo-35513-avoid-crash-while-processing-incor.patch
 Patch34: 0001-fedora-gcc-4.6.1.patch
 Patch35: 0001-make-sure-we-stay-in-array-bounds.patch
+Patch36: 0001-let-Qt-call-XInitThreads-so-that-it-knows-it-s-been-.patch
 
 %{!?python_sitearch: %global python_sitearch %(%{__python} -c "from distutils.sysconfig import get_python_lib; print(get_python_lib(1))")}
 %define instdir %{_libdir}
@@ -804,6 +805,7 @@ mv -f redhat.soc extras/source/palettes/standard.soc
 %patch33 -p1 -b .fdo35513-avoid-crash-while-processing-incor.patch
 %patch34 -p1 -b .fedora-gcc-4.6.1.patch
 %patch35 -p1 -b .make-sure-we-stay-in-array-bounds.patch
+%patch36 -p1 -b .let-Qt-call-XInitThreads-so-that-it-knows-it-s-been-.patch
 
 # these are horribly incomplete--empty translations and copied english
 # strings with spattering of translated strings
@@ -2106,6 +2108,9 @@ update-desktop-database %{_datadir}/applications &> /dev/null || :
 %{basisinstdir}/program/kde-open-url
 
 %changelog
+* Fri Oct 21 2011 Caolán McNamara <caolanm at redhat.com> - 3.4.3.2-13
+- Resolves: rhbz#747356 let Qt call XInitThreads
+
 * Wed Oct 19 2011 Caolán McNamara <caolanm at redhat.com> - 3.4.3.2-12
 - Related: rhbz#743750 addXineramaScreenUnique issue
 


More information about the scm-commits mailing list