summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSeth R. Johnson <johnsonsr@ornl.gov>2020-02-17 18:21:31 -0500
committerGitHub <noreply@github.com>2020-02-17 17:21:31 -0600
commit921cb6c860741574a0be8dd37542b5c306934d78 (patch)
tree60c1b84497e139721af7636e584aa86fc0834678
parentccbdd3c902c3be62739bb72d7a15d9a3d4ad73b4 (diff)
downloadspack-921cb6c860741574a0be8dd37542b5c306934d78.tar.gz
spack-921cb6c860741574a0be8dd37542b5c306934d78.tar.bz2
spack-921cb6c860741574a0be8dd37542b5c306934d78.tar.xz
spack-921cb6c860741574a0be8dd37542b5c306934d78.zip
Qt4 patches (#15025)
* Fix patch applicability * Combine patches for missing qt3 headers and remove krell variant The variant should have always been applied. * Restrict QT patches to actual applicable versions/situations - I researched the associated patches so now their `when=` should more closely match when they're actually needed. - I sorted the patch order so they're grouped by version requirement - I renamed the patches so they're listed by version requirements
-rw-r--r--var/spack/repos/builtin/packages/openspeedshop/package.py2
-rw-r--r--var/spack/repos/builtin/packages/qt/package.py77
-rw-r--r--var/spack/repos/builtin/packages/qt/qt3-accept.patch (renamed from var/spack/repos/builtin/packages/qt/qt3accept.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt3-headers.patch (renamed from var/spack/repos/builtin/packages/qt/qt3ptrdiff.patch)10
-rw-r--r--var/spack/repos/builtin/packages/qt/qt3krell.patch10
-rw-r--r--var/spack/repos/builtin/packages/qt/qt4-87-configure-gcc.patch (renamed from var/spack/repos/builtin/packages/qt/qt4-gcc-and-webkit-487.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt4-asm-volatile.patch (renamed from var/spack/repos/builtin/packages/qt/qt4-gcc8.3-asm-volatile-fix.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt4-configure-gcc.patch (renamed from var/spack/repos/builtin/packages/qt/qt4-gcc-and-webkit.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt4-pcre.patch (renamed from var/spack/repos/builtin/packages/qt/qt4-pcre-include-conflict.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt4-qforeach.patch (renamed from var/spack/repos/builtin/packages/qt/qt4-gcc9-qforeach.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt5-11-intel-overflow.patch (renamed from var/spack/repos/builtin/packages/qt/qt5_11-intel-overflow.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt5-12-intel-overflow.patch (renamed from var/spack/repos/builtin/packages/qt/qt5_12-intel-overflow.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt5-8-framework.patch (renamed from var/spack/repos/builtin/packages/qt/QTBUG-57656.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt5-8-freetype.patch (renamed from var/spack/repos/builtin/packages/qt/QTBUG-58038.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt5-asm-volatile.patch (renamed from var/spack/repos/builtin/packages/qt/qt5-gcc8.3-asm-volatile-fix.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt5-btn_trigger_happy.patch (renamed from var/spack/repos/builtin/packages/qt/btn_trigger_happy.patch)0
-rw-r--r--var/spack/repos/builtin/packages/qt/qt5-restore-pc-files.patch (renamed from var/spack/repos/builtin/packages/qt/restore-pc-files.patch)0
17 files changed, 42 insertions, 57 deletions
diff --git a/var/spack/repos/builtin/packages/openspeedshop/package.py b/var/spack/repos/builtin/packages/openspeedshop/package.py
index 88ac7e3547..56f9b0c689 100644
--- a/var/spack/repos/builtin/packages/openspeedshop/package.py
+++ b/var/spack/repos/builtin/packages/openspeedshop/package.py
@@ -96,7 +96,7 @@ class Openspeedshop(CMakePackage):
depends_on("libxml2")
- depends_on("qt@3.3.8b+krellpatch", when='gui=qt3')
+ depends_on("qt@3", when='gui=qt3')
# Dependencies for the openspeedshop cbtf packages.
depends_on("cbtf@develop", when='@develop', type=('build', 'link', 'run'))
diff --git a/var/spack/repos/builtin/packages/qt/package.py b/var/spack/repos/builtin/packages/qt/package.py
index 3753f4166c..4e6af88bb7 100644
--- a/var/spack/repos/builtin/packages/qt/package.py
+++ b/var/spack/repos/builtin/packages/qt/package.py
@@ -47,10 +47,6 @@ class Qt(Package):
version('4.8.5', sha256='eb728f8268831dc4373be6403b7dd5d5dde03c169ad6882f9a8cb560df6aa138')
version('3.3.8b', sha256='1b7a1ff62ec5a9cb7a388e2ba28fda6f960b27f27999482ebeceeadb72ac9f6e')
- # Add patch for compile issues with qt3 found with use in the
- # OpenSpeedShop project
- variant('krellpatch', default=False,
- description="Build with openspeedshop based patch.")
variant('gtk', default=False,
description="Build with gtkplus.")
variant('webkit', default=False,
@@ -76,55 +72,44 @@ class Qt(Package):
variant('freetype', default='spack', description='Freetype2 support',
values=('spack', 'qt', 'none'), multi=False)
- # fix installation of pkgconfig files
- # see https://github.com/Homebrew/homebrew-core/pull/5951
- patch('restore-pc-files.patch', when='@5.9:5.11 platform=darwin')
+ # Patches for qt@3
+ patch('qt3-accept.patch', when='@3')
+ patch('qt3-headers.patch', when='@3')
- patch('qt3accept.patch', when='@3.3.8b')
- patch('qt3krell.patch', when='@3.3.8b+krellpatch')
- patch('qt3ptrdiff.patch', when='@3.3.8b')
-
- # see https://bugreports.qt.io/browse/QTBUG-57656
- patch('QTBUG-57656.patch', when='@5.8.0')
- # see https://bugreports.qt.io/browse/QTBUG-58038
- patch('QTBUG-58038.patch', when='@5.8.0')
-
- # https://github.com/xboxdrv/xboxdrv/issues/188
- patch('btn_trigger_happy.patch', when='@5.7.0:')
+ # Patches for qt@4
+ patch('qt4-configure-gcc.patch', when='@4:4.8.6 %gcc')
+ patch('qt4-87-configure-gcc.patch', when='@4.8.7 %gcc')
+ patch('qt4-tools.patch', when='@4+tools')
+ patch('qt4-mac.patch', when='@4.8.7 platform=darwin')
+ # https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=925811
+ patch("qt4-qforeach.patch", when="@4 %gcc@9:")
+ # Patches for qt@4:
# https://github.com/spack/spack/issues/1517
+ patch('qt4-pcre.patch', when='@4')
patch('qt5-pcre.patch', when='@5:')
-
- patch('qt4-pcre-include-conflict.patch', when='@4.8.6')
- patch('qt4-tools.patch', when='@4+tools')
- if not MACOS_VERSION:
- # Allow Qt's configure script to build the webkit option with more
- # recent versions of gcc.
- # https://github.com/spack/spack/issues/9205
- # https://github.com/spack/spack/issues/9209
- patch('qt4-gcc-and-webkit.patch', when='@4:4.8.6')
- patch('qt4-gcc-and-webkit-487.patch', when='@4.8.7')
- else:
- patch('qt4-mac.patch', when='@4.8.7')
-
- # Fix build failure with newer versions of GCC
+ # https://bugreports.qt.io/browse/QTBUG-74196
+ # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89585
+ patch('qt4-asm-volatile.patch', when='@4')
+ patch('qt5-asm-volatile.patch', when='@5.0.0:5.12.1')
+
+ # Patches for qt@5
+ # https://bugreports.qt.io/browse/QTBUG-74219
+ patch('qt5-btn_trigger_happy.patch', when='@5.7:5.12')
+ # https://bugreports.qt.io/browse/QTBUG-57656
+ patch('qt5-8-framework.patch', when='@5.8.0 +framework')
+ # https://bugreports.qt.io/browse/QTBUG-58038
+ patch('qt5-8-freetype.patch', when='@5.8.0 freetype=spack')
+ # https://codereview.qt-project.org/c/qt/qtbase/+/245425
patch('https://github.com/qt/qtbase/commit/a52d7861edfb5956de38ba80015c4dd0b596259b.patch',
sha256='c49b228c27e3ad46ec3af4bac0e9985af5b5b28760f238422d32e14f98e49b1e',
working_dir='qtbase',
when='@5.10:5.12.0 %gcc@9:')
-
- # Fix build of QT4 with GCC 9
- # https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=925811
- patch("qt4-gcc9-qforeach.patch", when="@4:4.999 %gcc@9")
-
- # https://bugreports.qt.io/browse/QTBUG-74196
- # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89585
- patch('qt4-gcc8.3-asm-volatile-fix.patch', when='@4')
- patch('qt5-gcc8.3-asm-volatile-fix.patch', when='@5.0.0:5.12.1')
-
- # patch overflow builtins
- patch('qt5_11-intel-overflow.patch', when='@5.11')
- patch('qt5_12-intel-overflow.patch', when='@5.12:')
+ # https://github.com/Homebrew/homebrew-core/pull/5951
+ patch('qt5-restore-pc-files.patch', when='@5.9:5.11 platform=darwin')
+ # https://github.com/spack/spack/issues/14400
+ patch('qt5-11-intel-overflow.patch', when='@5.11 %intel')
+ patch('qt5-12-intel-overflow.patch', when='@5.12:5.14.0 %intel')
# Build-only dependencies
depends_on("pkgconfig", type='build')
@@ -149,7 +134,7 @@ class Qt(Package):
depends_on("libpng@1.2.57", when='@3')
depends_on("pcre+multibyte", when='@5.0:5.8')
depends_on("inputproto", when='@:5.8')
- depends_on("openssl@:1.0.999", when='@:5.9+ssl~krellpatch')
+ depends_on("openssl@:1.0.999", when='@:5.9+ssl')
depends_on("glib", when='@4:')
depends_on("libpng", when='@4:')
diff --git a/var/spack/repos/builtin/packages/qt/qt3accept.patch b/var/spack/repos/builtin/packages/qt/qt3-accept.patch
index 32dfb9df08..32dfb9df08 100644
--- a/var/spack/repos/builtin/packages/qt/qt3accept.patch
+++ b/var/spack/repos/builtin/packages/qt/qt3-accept.patch
diff --git a/var/spack/repos/builtin/packages/qt/qt3ptrdiff.patch b/var/spack/repos/builtin/packages/qt/qt3-headers.patch
index 5043e4b5a8..52b133b571 100644
--- a/var/spack/repos/builtin/packages/qt/qt3ptrdiff.patch
+++ b/var/spack/repos/builtin/packages/qt/qt3-headers.patch
@@ -18,3 +18,13 @@
#endif
//#define QT_CHECK_VALUELIST_RANGE
+--- qt-x11-free-3.3.8b/src/tools/qvaluevector.h 2008-01-15 13:09:13.000000000 -0600
++++ qt-x11-free-3.3.8b-fixes/src/tools/qvaluevector.h 2015-07-08 15:47:34.758565247 -0500
+@@ -47,6 +47,7 @@
+
+ #ifndef QT_NO_STL
+ #include <vector>
++#include <cstddef>
+ #endif
+
+ template <class T>
diff --git a/var/spack/repos/builtin/packages/qt/qt3krell.patch b/var/spack/repos/builtin/packages/qt/qt3krell.patch
deleted file mode 100644
index cee94b3cbc..0000000000
--- a/var/spack/repos/builtin/packages/qt/qt3krell.patch
+++ /dev/null
@@ -1,10 +0,0 @@
---- qt-x11-free-3.3.8b/src/tools/qvaluevector.h 2008-01-15 13:09:13.000000000 -0600
-+++ qt-x11-free-3.3.8b-fixes/src/tools/qvaluevector.h 2015-07-08 15:47:34.758565247 -0500
-@@ -47,6 +47,7 @@
-
- #ifndef QT_NO_STL
- #include <vector>
-+#include <cstddef>
- #endif
-
- template <class T>
diff --git a/var/spack/repos/builtin/packages/qt/qt4-gcc-and-webkit-487.patch b/var/spack/repos/builtin/packages/qt/qt4-87-configure-gcc.patch
index ee3a6cd524..ee3a6cd524 100644
--- a/var/spack/repos/builtin/packages/qt/qt4-gcc-and-webkit-487.patch
+++ b/var/spack/repos/builtin/packages/qt/qt4-87-configure-gcc.patch
diff --git a/var/spack/repos/builtin/packages/qt/qt4-gcc8.3-asm-volatile-fix.patch b/var/spack/repos/builtin/packages/qt/qt4-asm-volatile.patch
index e3d3cae1f7..e3d3cae1f7 100644
--- a/var/spack/repos/builtin/packages/qt/qt4-gcc8.3-asm-volatile-fix.patch
+++ b/var/spack/repos/builtin/packages/qt/qt4-asm-volatile.patch
diff --git a/var/spack/repos/builtin/packages/qt/qt4-gcc-and-webkit.patch b/var/spack/repos/builtin/packages/qt/qt4-configure-gcc.patch
index 38a1c350f3..38a1c350f3 100644
--- a/var/spack/repos/builtin/packages/qt/qt4-gcc-and-webkit.patch
+++ b/var/spack/repos/builtin/packages/qt/qt4-configure-gcc.patch
diff --git a/var/spack/repos/builtin/packages/qt/qt4-pcre-include-conflict.patch b/var/spack/repos/builtin/packages/qt/qt4-pcre.patch
index 854e564bfb..854e564bfb 100644
--- a/var/spack/repos/builtin/packages/qt/qt4-pcre-include-conflict.patch
+++ b/var/spack/repos/builtin/packages/qt/qt4-pcre.patch
diff --git a/var/spack/repos/builtin/packages/qt/qt4-gcc9-qforeach.patch b/var/spack/repos/builtin/packages/qt/qt4-qforeach.patch
index ae54cdbc1e..ae54cdbc1e 100644
--- a/var/spack/repos/builtin/packages/qt/qt4-gcc9-qforeach.patch
+++ b/var/spack/repos/builtin/packages/qt/qt4-qforeach.patch
diff --git a/var/spack/repos/builtin/packages/qt/qt5_11-intel-overflow.patch b/var/spack/repos/builtin/packages/qt/qt5-11-intel-overflow.patch
index 5bb0b1e03e..5bb0b1e03e 100644
--- a/var/spack/repos/builtin/packages/qt/qt5_11-intel-overflow.patch
+++ b/var/spack/repos/builtin/packages/qt/qt5-11-intel-overflow.patch
diff --git a/var/spack/repos/builtin/packages/qt/qt5_12-intel-overflow.patch b/var/spack/repos/builtin/packages/qt/qt5-12-intel-overflow.patch
index bfe5c8e4f3..bfe5c8e4f3 100644
--- a/var/spack/repos/builtin/packages/qt/qt5_12-intel-overflow.patch
+++ b/var/spack/repos/builtin/packages/qt/qt5-12-intel-overflow.patch
diff --git a/var/spack/repos/builtin/packages/qt/QTBUG-57656.patch b/var/spack/repos/builtin/packages/qt/qt5-8-framework.patch
index b1ba47fa00..b1ba47fa00 100644
--- a/var/spack/repos/builtin/packages/qt/QTBUG-57656.patch
+++ b/var/spack/repos/builtin/packages/qt/qt5-8-framework.patch
diff --git a/var/spack/repos/builtin/packages/qt/QTBUG-58038.patch b/var/spack/repos/builtin/packages/qt/qt5-8-freetype.patch
index c8d71105ab..c8d71105ab 100644
--- a/var/spack/repos/builtin/packages/qt/QTBUG-58038.patch
+++ b/var/spack/repos/builtin/packages/qt/qt5-8-freetype.patch
diff --git a/var/spack/repos/builtin/packages/qt/qt5-gcc8.3-asm-volatile-fix.patch b/var/spack/repos/builtin/packages/qt/qt5-asm-volatile.patch
index c49fe3f341..c49fe3f341 100644
--- a/var/spack/repos/builtin/packages/qt/qt5-gcc8.3-asm-volatile-fix.patch
+++ b/var/spack/repos/builtin/packages/qt/qt5-asm-volatile.patch
diff --git a/var/spack/repos/builtin/packages/qt/btn_trigger_happy.patch b/var/spack/repos/builtin/packages/qt/qt5-btn_trigger_happy.patch
index e6a27d5fab..e6a27d5fab 100644
--- a/var/spack/repos/builtin/packages/qt/btn_trigger_happy.patch
+++ b/var/spack/repos/builtin/packages/qt/qt5-btn_trigger_happy.patch
diff --git a/var/spack/repos/builtin/packages/qt/restore-pc-files.patch b/var/spack/repos/builtin/packages/qt/qt5-restore-pc-files.patch
index 35fbaddd43..35fbaddd43 100644
--- a/var/spack/repos/builtin/packages/qt/restore-pc-files.patch
+++ b/var/spack/repos/builtin/packages/qt/qt5-restore-pc-files.patch