diff mbox

[1/2] Qt5: update to 5.2.1

Message ID trinity-66f5431b-92f8-44b8-9cb0-1f564a6cda9f-1391694043383@3capp-gmx-bs23
State Superseded
Headers show

Commit Message

Peter Kümmel Feb. 6, 2014, 1:40 p.m. UTC

Comments

Thomas Petazzoni Feb. 6, 2014, 3:27 p.m. UTC | #1
Dear Peter Kuemmel,

On Thu, 6 Feb 2014 14:40:43 +0100 (CET), Peter Kuemmel wrote:

> From 315027aef2dec428857bcd0699ea60a7bcc80a8c Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Peter=20K=C3=BCmmel?= <kuemmel@coffeelogic.de>
> Date: Wed, 5 Feb 2014 20:49:47 +0100
> Subject: [PATCH 1/2] Qt5: update to 5.2.1
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit

As explained in the documentation, could you use "git send-email" to
send your patches? For the moment, they are sent as attached files,
which is very unpractical for review.

Thanks a lot for these contributions!

Thomas
Peter Kümmel Feb. 6, 2014, 3:44 p.m. UTC | #2
> Von: "Thomas Petazzoni" <thomas.petazzoni@free-electrons.com>
> 
> As explained in the documentation, could you use "git send-email" to

Sorry, I thought using send-email is just optional.

So someone else interested should put it into the queue,
but don't use my signed-off patches.

The qttools patch needs a change:

+	cp -dpf $(STAGING_DIR)/usr/lib/libQt5Help.so.* $(TARGET_DIR)/usr/lib
+	cp -dpf $(STAGING_DIR)/usr/lib/libQt5CLucene.so.* $(TARGET_DIR)/usr/lib

Thanks,
Peter

> send your patches? For the moment, they are sent as attached files,
> which is very unpractical for review.
> 
> Thanks a lot for these contributions!
> 
> Thomas
> -- 
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
>
Thomas Petazzoni Feb. 6, 2014, 3:50 p.m. UTC | #3
Dear Peter Kuemmel,

On Thu, 6 Feb 2014 16:44:28 +0100 (CET), Peter Kuemmel wrote:

> > As explained in the documentation, could you use "git send-email" to
> 
> Sorry, I thought using send-email is just optional.

It is in some sense optional, if you do send patches "the right way".
But the easiest solution to send them "the right way" is to use git
send-email.

> So someone else interested should put it into the queue,
> but don't use my signed-off patches.

Unfortunately, this often doesn't work: once someone sends patches,
there are review comments, and the submitter often has to make a few
changes, and resubmit. So if someone else picks up your patches, he
will have to do this work, while as the original author of the patches,
it would be much easier if you did this work.

Those contributions are very useful to Buildroot, so it would very
appreciated if you could go through the contribution process
completely. I know it takes a bit of time, but it is a necessary
process to ensure that the quality of patches merged is up to the
standards the community has defined.

Thanks a lot!

Thomas
diff mbox

Patch

From 315027aef2dec428857bcd0699ea60a7bcc80a8c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Peter=20K=C3=BCmmel?= <kuemmel@coffeelogic.de>
Date: Wed, 5 Feb 2014 20:49:47 +0100
Subject: [PATCH 1/2] Qt5: update to 5.2.1
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Peter Kümmel <kuemmel@coffeelogic.de>
---
 package/qt5/qt5.mk                                 |   2 +-
 ...on-t-require-GL-when-build-with-no-opengl.patch |  28 ++++
 .../qt5base-0001-Fix-build-MIPS64-bigendian.patch  |  51 -------
 ...-detect-posix_fallocate-at-configure-time.patch | 167 ---------------------
 4 files changed, 29 insertions(+), 219 deletions(-)
 create mode 100644 package/qt5/qt5base/qt5base-0001-CMake-don-t-require-GL-when-build-with-no-opengl.patch
 delete mode 100644 package/qt5/qt5base/qt5base-0001-Fix-build-MIPS64-bigendian.patch
 delete mode 100644 package/qt5/qt5base/qt5base-0007-detect-posix_fallocate-at-configure-time.patch

diff --git a/package/qt5/qt5.mk b/package/qt5/qt5.mk
index 6b05334..0ce6cf5 100644
--- a/package/qt5/qt5.mk
+++ b/package/qt5/qt5.mk
@@ -1,5 +1,5 @@ 
 QT5_VERSION_MAJOR = 5.2
-QT5_VERSION = $(QT5_VERSION_MAJOR).0
+QT5_VERSION = $(QT5_VERSION_MAJOR).1
 QT5_SITE = http://download.qt-project.org/official_releases/qt/$(QT5_VERSION_MAJOR)/$(QT5_VERSION)/submodules/
 include $(sort $(wildcard package/qt5/*/*.mk))
 
diff --git a/package/qt5/qt5base/qt5base-0001-CMake-don-t-require-GL-when-build-with-no-opengl.patch b/package/qt5/qt5base/qt5base-0001-CMake-don-t-require-GL-when-build-with-no-opengl.patch
new file mode 100644
index 0000000..e04bcc4
--- /dev/null
+++ b/package/qt5/qt5base/qt5base-0001-CMake-don-t-require-GL-when-build-with-no-opengl.patch
@@ -0,0 +1,28 @@ 
+From 02edc30764935d64f0994a9c592ca729f1c37410 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Peter=20K=C3=BCmmel?= <syntheticpp@gmx.net>
+Date: Fri, 31 Jan 2014 22:28:19 +0100
+Subject: [PATCH] CMake: don't require GL when build with -no-opengl
+
+Task-number: QTBUG-36509
+Change-Id: I03451527ab91754d87ff2a43985952c7c7641826
+Reviewed-by: Stephen Kelly <stephen.kelly@kdab.com>
+---
+ src/gui/gui.pro | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/gui/gui.pro b/src/gui/gui.pro
+index 3203b41..f4c35a3 100644
+--- a/src/gui/gui.pro
++++ b/src/gui/gui.pro
+@@ -81,7 +81,7 @@ contains(QT_CONFIG, angle) {
+         !isEmpty(QMAKE_LIBDIR_OPENGL_ES2): CMAKE_OPENGL_LIBDIR = $$cmakePortablePaths($$QMAKE_LIBDIR_OPENGL_ES2)
+         CMAKE_GL_HEADER_NAME = GLES2/gl2.h
+         CMAKE_QT_OPENGL_IMPLEMENTATION = GLESv2
+-    } else {
++    } else:contains(QT_CONFIG, opengl) {
+         !isEmpty(QMAKE_INCDIR_OPENGL): CMAKE_GL_INCDIRS = $$cmakeTargetPaths($$QMAKE_INCDIR_OPENGL)
+         CMAKE_OPENGL_INCDIRS = $$cmakePortablePaths($$QMAKE_INCDIR_OPENGL)
+         CMAKE_OPENGL_LIBS = $$cmakeProcessLibs($$QMAKE_LIBS_OPENGL)
+-- 
+1.8.3.2
+
diff --git a/package/qt5/qt5base/qt5base-0001-Fix-build-MIPS64-bigendian.patch b/package/qt5/qt5base/qt5base-0001-Fix-build-MIPS64-bigendian.patch
deleted file mode 100644
index 00dcd48..0000000
--- a/package/qt5/qt5base/qt5base-0001-Fix-build-MIPS64-bigendian.patch
+++ /dev/null
@@ -1,51 +0,0 @@ 
-qt5base: Fix build error on 64bit BigEndian platforms
-Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
-
-From cea101bd10cc158b97d5fbf45b33dad9b649c08b Mon Sep 17 00:00:00 2001
-From: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
-Date: Mon, 2 Dec 2013 13:59:34 +0000
-Subject: [PATCH] qtbase: Fix build error on 64bit BigEndian platforms
-
-The functions fromBytesToWord() and fromWordToBytes() are called when
-building on 64bit BigEndian platforms. It fails because those
-functions are disabled on the source code. Enabling those functions for
-64bit BigEndian platforms fixes the problem.
-
-Task-number: QTBUG-35228
-
-Change-Id: I5ccacd4fb5051df05f67c8da879b3a9e49953861
-Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
-Reviewed-by: Giuseppe D'Angelo <giuseppe.dangelo@kdab.com>
-Reviewed-by: Richard J. Moore <rich@kde.org>
----
- src/3rdparty/sha3/KeccakF-1600-opt64.c | 4 ++--
- 1 file changed, 2 insertions(+), 2 deletions(-)
- mode change 100755 => 100644 src/3rdparty/sha3/KeccakF-1600-opt64.c
-
-diff --git a/src/3rdparty/sha3/KeccakF-1600-opt64.c b/src/3rdparty/sha3/KeccakF-1600-opt64.c
-old mode 100755
-new mode 100644
-index 7bd442e..a547bb5
---- a/src/3rdparty/sha3/KeccakF-1600-opt64.c
-+++ b/src/3rdparty/sha3/KeccakF-1600-opt64.c
-@@ -328,7 +328,7 @@ static void KeccakPermutation(unsigned char *state)
-     KeccakPermutationOnWords((UINT64*)state);
- }
- 
--#if 0 // Unused in the Qt configuration
-+#if (PLATFORM_BYTE_ORDER == IS_BIG_ENDIAN)
- static void fromBytesToWord(UINT64 *word, const UINT8 *bytes)
- {
-     unsigned int i;
-@@ -449,7 +449,7 @@ static void KeccakAbsorb(unsigned char *state, const unsigned char *data, unsign
- #endif
- }
- 
--#if 0 // Unused in the Qt configuration
-+#if (PLATFORM_BYTE_ORDER == IS_BIG_ENDIAN)
- static void fromWordToBytes(UINT8 *bytes, const UINT64 word)
- {
-     unsigned int i;
--- 
-1.8.5.1
-
diff --git a/package/qt5/qt5base/qt5base-0007-detect-posix_fallocate-at-configure-time.patch b/package/qt5/qt5base/qt5base-0007-detect-posix_fallocate-at-configure-time.patch
deleted file mode 100644
index b0011f6..0000000
--- a/package/qt5/qt5base/qt5base-0007-detect-posix_fallocate-at-configure-time.patch
+++ /dev/null
@@ -1,167 +0,0 @@ 
-From 46e632e2555b6b8abe6b8d8ad7d255e27c128e25 Mon Sep 17 00:00:00 2001
-From: =?UTF-8?q?Fatih=20A=C5=9F=C4=B1c=C4=B1?= <fatih.asici@gmail.com>
-Date: Mon, 11 Nov 2013 17:04:46 +0200
-Subject: [PATCH] Detect posix_fallocate at configure time
-
-Testing feature macros is not enough for uclibc. Fixes build of the built-in
-sqlite3 with uclibc <= 0.9.33.2. Later versions will have posix_fallocate().
-
-Change-Id: I918a52777ac63624635802221effc6b86fa2269c
-Signed-off-by: Fatih Aşıcı <fatih.asici@gmail.com>
-Reported-Upstream: https://codereview.qt-project.org/70935
----
- .../unix/posix_fallocate/posix_fallocate.cpp       | 53 ++++++++++++++++++++++
- .../unix/posix_fallocate/posix_fallocate.pro       |  2 +
- configure                                          | 14 ++++++
- src/3rdparty/sqlite.pri                            |  1 +
- src/3rdparty/sqlite/sqlite3.c                      |  7 ---
- 5 files changed, 70 insertions(+), 7 deletions(-)
- create mode 100644 config.tests/unix/posix_fallocate/posix_fallocate.cpp
- create mode 100644 config.tests/unix/posix_fallocate/posix_fallocate.pro
-
-diff --git a/config.tests/unix/posix_fallocate/posix_fallocate.cpp b/config.tests/unix/posix_fallocate/posix_fallocate.cpp
-new file mode 100644
-index 0000000..5acd45a
---- /dev/null
-+++ b/config.tests/unix/posix_fallocate/posix_fallocate.cpp
-@@ -0,0 +1,53 @@
-+/****************************************************************************
-+**
-+** Copyright (C) 2013 Digia Plc and/or its subsidiary(-ies).
-+** Contact: http://www.qt-project.org/legal
-+**
-+** This file is part of the config.tests of the Qt Toolkit.
-+**
-+** $QT_BEGIN_LICENSE:LGPL$
-+** Commercial License Usage
-+** Licensees holding valid commercial Qt licenses may use this file in
-+** accordance with the commercial license agreement provided with the
-+** Software or, alternatively, in accordance with the terms contained in
-+** a written agreement between you and Digia.  For licensing terms and
-+** conditions see http://qt.digia.com/licensing.  For further information
-+** use the contact form at http://qt.digia.com/contact-us.
-+**
-+** GNU Lesser General Public License Usage
-+** Alternatively, this file may be used under the terms of the GNU Lesser
-+** General Public License version 2.1 as published by the Free Software
-+** Foundation and appearing in the file LICENSE.LGPL included in the
-+** packaging of this file.  Please review the following information to
-+** ensure the GNU Lesser General Public License version 2.1 requirements
-+** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html.
-+**
-+** In addition, as a special exception, Digia gives you certain additional
-+** rights.  These rights are described in the Digia Qt LGPL Exception
-+** version 1.1, included in the file LGPL_EXCEPTION.txt in this package.
-+**
-+** GNU General Public License Usage
-+** Alternatively, this file may be used under the terms of the GNU
-+** General Public License version 3.0 as published by the Free Software
-+** Foundation and appearing in the file LICENSE.GPL included in the
-+** packaging of this file.  Please review the following information to
-+** ensure the GNU General Public License version 3.0 requirements will be
-+** met: http://www.gnu.org/copyleft/gpl.html.
-+**
-+**
-+** $QT_END_LICENSE$
-+**
-+****************************************************************************/
-+
-+#include <fcntl.h>
-+
-+int main(int, char **)
-+{
-+#if _XOPEN_SOURCE >= 600 || _POSIX_C_SOURCE >= 200112L
-+    return ::posix_fallocate(0, 0, 0);
-+#else
-+#  error posix_fallocate not available
-+    // MIPSpro doesn't understand #error, so force a compiler error
-+    force_compiler_error = true;
-+#endif
-+}
-diff --git a/config.tests/unix/posix_fallocate/posix_fallocate.pro b/config.tests/unix/posix_fallocate/posix_fallocate.pro
-new file mode 100644
-index 0000000..f01b15f
---- /dev/null
-+++ b/config.tests/unix/posix_fallocate/posix_fallocate.pro
-@@ -0,0 +1,2 @@
-+SOURCES = posix_fallocate.cpp
-+CONFIG -= qt dylib
-diff --git a/configure b/configure
-index 81e2a93..b67ed89 100755
---- a/configure
-+++ b/configure
-@@ -968,6 +968,7 @@ CFG_MIPS_DSP=auto
- CFG_MIPS_DSPR2=auto
- CFG_CLOCK_GETTIME=auto
- CFG_CLOCK_MONOTONIC=auto
-+CFG_POSIX_FALLOCATE=auto
- CFG_MREMAP=auto
- CFG_GETADDRINFO=auto
- CFG_IPV6IFNAME=auto
-@@ -5653,6 +5654,15 @@ elif [ "$CFG_CLOCK_GETTIME" = "no" ]; then
-     CFG_CLOCK_MONOTONIC=no
- fi
- 
-+# detect posix_fallocate
-+if [ "$CFG_POSIX_FALLOCATE" = "auto" ]; then
-+    if compileTest unix/posix_fallocate "posix_fallocate"; then
-+        CFG_POSIX_FALLOCATE=yes
-+    else
-+        CFG_POSIX_FALLOCATE=no
-+    fi
-+fi
-+
- # detect mremap
- if [ "$CFG_MREMAP" = "auto" ]; then
-     if compileTest unix/mremap "mremap"; then
-@@ -5970,6 +5980,9 @@ fi
- if [ "$CFG_CLOCK_MONOTONIC" = "yes" ]; then
-     QT_CONFIG="$QT_CONFIG clock-monotonic"
- fi
-+if [ "$CFG_POSIX_FALLOCATE" = "yes" ]; then
-+    QT_CONFIG="$QT_CONFIG posix_fallocate"
-+fi
- if [ "$CFG_MREMAP" = "yes" ]; then
-     QT_CONFIG="$QT_CONFIG mremap"
- fi
-@@ -6449,6 +6462,7 @@ QMakeVar set sql-plugins "$SQL_PLUGINS"
- [ "$CFG_GLIB" != "yes" ]     && QCONFIG_FLAGS="$QCONFIG_FLAGS QT_NO_GLIB"
- [ "$CFG_QGTKSTYLE" != "yes" ] && QCONFIG_FLAGS="$QCONFIG_FLAGS QT_NO_STYLE_GTK"
- [ "$CFG_CLOCK_MONOTONIC" = "no" ] && QCONFIG_FLAGS="$QCONFIG_FLAGS QT_NO_CLOCK_MONOTONIC"
-+[ "$CFG_POSIX_FALLOCATE" = "no" ] && QCONFIG_FLAGS="$QCONFIG_FLAGS QT_NO_POSIX_FALLOCATE"
- [ "$CFG_MREMAP" = "no" ]     && QCONFIG_FLAGS="$QCONFIG_FLAGS QT_NO_MREMAP"
- [ "$CFG_GETADDRINFO" = "no" ]&& QCONFIG_FLAGS="$QCONFIG_FLAGS QT_NO_GETADDRINFO"
- [ "$CFG_IPV6IFNAME" = "no" ] && QCONFIG_FLAGS="$QCONFIG_FLAGS QT_NO_IPV6IFNAME"
-diff --git a/src/3rdparty/sqlite.pri b/src/3rdparty/sqlite.pri
-index 58d4ddd..072502c 100644
---- a/src/3rdparty/sqlite.pri
-+++ b/src/3rdparty/sqlite.pri
-@@ -1,6 +1,7 @@
- CONFIG(release, debug|release):DEFINES *= NDEBUG
- DEFINES += SQLITE_OMIT_LOAD_EXTENSION SQLITE_OMIT_COMPLETE SQLITE_ENABLE_FTS3 SQLITE_ENABLE_FTS3_PARENTHESIS SQLITE_ENABLE_RTREE
- !contains(CONFIG, largefile):DEFINES += SQLITE_DISABLE_LFS
-+contains(QT_CONFIG, posix_fallocate):DEFINES += HAVE_POSIX_FALLOCATE=1
- winrt: DEFINES += SQLITE_OS_WINRT
- INCLUDEPATH +=  $$PWD/sqlite
- SOURCES +=      $$PWD/sqlite/sqlite3.c
-diff --git a/src/3rdparty/sqlite/sqlite3.c b/src/3rdparty/sqlite/sqlite3.c
-index 03fa649..1ae9be2 100644
---- a/src/3rdparty/sqlite/sqlite3.c
-+++ b/src/3rdparty/sqlite/sqlite3.c
-@@ -22935,13 +22935,6 @@ SQLITE_PRIVATE const char *sqlite3OpcodeName(int i){
- */
- #if SQLITE_OS_UNIX              /* This file is used on unix only */
- 
--/* Use posix_fallocate() if it is available
--*/
--#if !defined(HAVE_POSIX_FALLOCATE) \
--      && (_XOPEN_SOURCE >= 600 || _POSIX_C_SOURCE >= 200112L)
--# define HAVE_POSIX_FALLOCATE 1
--#endif
--
- /*
- ** There are various methods for file locking used for concurrency
- ** control:
--- 
-1.8.4.rc3
-
-- 
1.8.3.2