* [Buildroot] [PATCH/next v3 1/1] package/qt5/qt5base: fix statx support with glibc >= 2.28
@ 2018-08-13 19:42 Bernd Kuhls
2018-08-14 13:41 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: Bernd Kuhls @ 2018-08-13 19:42 UTC (permalink / raw)
To: buildroot
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v3: switched to upstream patch (Thomas)
v2: git-formatted patch (Thomas)
re-sent because it did not appear on patchworks
...0002-Fix-qmake-build-with-glibc-2.28.patch | 74 +++++++++++++++++++
1 file changed, 74 insertions(+)
create mode 100644 package/qt5/qt5base/5.11.1/0002-Fix-qmake-build-with-glibc-2.28.patch
diff --git a/package/qt5/qt5base/5.11.1/0002-Fix-qmake-build-with-glibc-2.28.patch b/package/qt5/qt5base/5.11.1/0002-Fix-qmake-build-with-glibc-2.28.patch
new file mode 100644
index 0000000000..6d8bb28992
--- /dev/null
+++ b/package/qt5/qt5base/5.11.1/0002-Fix-qmake-build-with-glibc-2.28.patch
@@ -0,0 +1,74 @@
+From 25feee4e061b99edab79503d81f5bd045c6c8e3d Mon Sep 17 00:00:00 2001
+From: Thiago Macieira <thiago.macieira@intel.com>
+Date: Tue, 7 Aug 2018 09:38:42 -0700
+Subject: [PATCH] Fix qmake build with glibc 2.28
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+We haven't yet run the configure checks to see if statx and renameat2
+are present in glibc, so this fails when we redefine the structures and
+functions.
+
+linux/stat.h:56:8: error: redefinition of 'struct statx_timestamp'
+bits/statx.h:25:8: note: previous definition of 'struct statx_timestamp'
+qfilesystemengine_unix.cpp:110:12: error: 'int renameat2(int, const char*, int, const char*, unsigned int)' was declared 'extern' and later 'static' [-fpermissive]
+
+Change-Id: Ia741b559c24d46c78fb2fffd1548a792d22e3368
+Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@qt.io>
+Reviewed-by: J?ri Valdmann <juri.valdmann@qt.io>
+
+Downloaded from upstream commit
+http://code.qt.io/cgit/qt/qtbase.git/commit/?id=25feee4e061b99edab79503d81f5bd045c6c8e3d
+
+Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
+---
+ src/corelib/global/qconfig-bootstrapped.h | 12 ++++++++++--
+ src/corelib/io/qfilesystemengine_unix.cpp | 2 +-
+ 2 files changed, 11 insertions(+), 3 deletions(-)
+
+diff --git a/src/corelib/global/qconfig-bootstrapped.h b/src/corelib/global/qconfig-bootstrapped.h
+index 3469ebe5e6..c5585ea32a 100644
+--- a/src/corelib/global/qconfig-bootstrapped.h
++++ b/src/corelib/global/qconfig-bootstrapped.h
+@@ -98,10 +98,18 @@
+ #define QT_NO_QOBJECT
+ #define QT_FEATURE_process -1
+ #define QT_FEATURE_regularexpression -1
+-#define QT_FEATURE_renameat2 -1
++#ifdef __GLIBC_PREREQ
++# define QT_FEATURE_renameat2 (__GLIBC_PREREQ(2, 28) ? 1 : -1)
++#else
++# define QT_FEATURE_renameat2 -1
++#endif
+ #define QT_FEATURE_sharedmemory -1
+ #define QT_FEATURE_slog2 -1
+-#define QT_FEATURE_statx -1
++#ifdef __GLIBC_PREREQ
++# define QT_FEATURE_statx (__GLIBC_PREREQ(2, 28) ? 1 : -1)
++#else
++# define QT_FEATURE_statx -1
++#endif
+ #define QT_FEATURE_syslog -1
+ #define QT_NO_SYSTEMLOCALE
+ #define QT_FEATURE_systemsemaphore -1
+diff --git a/src/corelib/io/qfilesystemengine_unix.cpp b/src/corelib/io/qfilesystemengine_unix.cpp
+index 0c9cdb8667..deb4a9f220 100644
+--- a/src/corelib/io/qfilesystemengine_unix.cpp
++++ b/src/corelib/io/qfilesystemengine_unix.cpp
+@@ -91,7 +91,6 @@ extern "C" NSString *NSTemporaryDirectory();
+ # include <sys/syscall.h>
+ # include <sys/sendfile.h>
+ # include <linux/fs.h>
+-# include <linux/stat.h>
+
+ // in case linux/fs.h is too old and doesn't define it:
+ #ifndef FICLONE
+@@ -112,6 +111,7 @@ static int renameat2(int oldfd, const char *oldpath, int newfd, const char *newp
+ # endif
+
+ # if !QT_CONFIG(statx) && defined(SYS_statx)
++# include <linux/stat.h>
+ static int statx(int dirfd, const char *pathname, int flag, unsigned mask, struct statx *statxbuf)
+ { return syscall(SYS_statx, dirfd, pathname, flag, mask, statxbuf); }
+ # elif !QT_CONFIG(statx) && !defined(SYS_statx)
--
2.18.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* [Buildroot] [PATCH/next v3 1/1] package/qt5/qt5base: fix statx support with glibc >= 2.28
2018-08-13 19:42 [Buildroot] [PATCH/next v3 1/1] package/qt5/qt5base: fix statx support with glibc >= 2.28 Bernd Kuhls
@ 2018-08-14 13:41 ` Thomas Petazzoni
2018-08-16 19:11 ` Bernd Kuhls
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2018-08-14 13:41 UTC (permalink / raw)
To: buildroot
Hello,
On Mon, 13 Aug 2018 21:42:06 +0200, Bernd Kuhls wrote:
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v3: switched to upstream patch (Thomas)
> v2: git-formatted patch (Thomas)
> re-sent because it did not appear on patchworks
Applied to next, thanks. Did you check if this issue also applies to Qt
5.6.3, which we still support ?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH/next v3 1/1] package/qt5/qt5base: fix statx support with glibc >= 2.28
2018-08-14 13:41 ` Thomas Petazzoni
@ 2018-08-16 19:11 ` Bernd Kuhls
2018-08-16 20:05 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: Bernd Kuhls @ 2018-08-16 19:11 UTC (permalink / raw)
To: buildroot
Am Tue, 14 Aug 2018 15:41:27 +0200 schrieb Thomas Petazzoni:
> Did you check if this issue also applies to Qt
> 5.6.3, which we still support ?
Hi Thomas,
because the qt5 dependency of vlc was my first ever contact with qt5 in
buildroot I did not notice that 5.6 can also be selected while 5.11 is
the default version. I think qt 5.6 does not need any patches for
glibc-2.28 because I just did a successful build with the current next
branch:
BR2_x86_64=y
BR2_x86_core_avx2=y
BR2_TOOLCHAIN_BUILDROOT_GLIBC=y
BR2_BINUTILS_VERSION_2_31_X=y
BR2_GCC_VERSION_8_X=y
BR2_TOOLCHAIN_BUILDROOT_CXX=y
BR2_INIT_SYSTEMD=y
# BR2_PACKAGE_BUSYBOX is not set
BR2_PACKAGE_FFMPEG=y
BR2_PACKAGE_FFMPEG_GPL=y
BR2_PACKAGE_FFMPEG_NONFREE=y
BR2_PACKAGE_FFMPEG_FFPLAY=y
BR2_PACKAGE_FFMPEG_FFPROBE=y
BR2_PACKAGE_FFMPEG_AVRESAMPLE=y
BR2_PACKAGE_FFMPEG_POSTPROC=y
BR2_PACKAGE_VLC=y
BR2_PACKAGE_MESA3D=y
BR2_PACKAGE_MESA3D_DRI_DRIVER_I965=y
BR2_PACKAGE_MESA3D_OPENGL_EGL=y
BR2_PACKAGE_MESA3D_OPENGL_ES=y
BR2_PACKAGE_QT5=y
BR2_PACKAGE_QT5_VERSION_5_6=y
BR2_PACKAGE_QT5BASE_WIDGETS=y
BR2_PACKAGE_QT5SVG=y
BR2_PACKAGE_XORG7=y
Quotes from vlc configure log:
checking for QT... yes
[...]
vlc aliases : cvlc rvlc qvlc
Regards, Bernd
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH/next v3 1/1] package/qt5/qt5base: fix statx support with glibc >= 2.28
2018-08-16 19:11 ` Bernd Kuhls
@ 2018-08-16 20:05 ` Thomas Petazzoni
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2018-08-16 20:05 UTC (permalink / raw)
To: buildroot
Hello,
On Thu, 16 Aug 2018 21:11:17 +0200, Bernd Kuhls wrote:
> because the qt5 dependency of vlc was my first ever contact with qt5 in
> buildroot I did not notice that 5.6 can also be selected while 5.11 is
> the default version. I think qt 5.6 does not need any patches for
> glibc-2.28 because I just did a successful build with the current next
> branch:
OK, thanks for verifying this!
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-08-16 20:05 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-08-13 19:42 [Buildroot] [PATCH/next v3 1/1] package/qt5/qt5base: fix statx support with glibc >= 2.28 Bernd Kuhls
2018-08-14 13:41 ` Thomas Petazzoni
2018-08-16 19:11 ` Bernd Kuhls
2018-08-16 20:05 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox