* [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation
@ 2017-05-17 21:15 Yann E. MORIN
2017-05-17 21:15 ` [Buildroot] [PATCH 1/2 v2] package/qt5base: install arch-specific configuration Yann E. MORIN
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Yann E. MORIN @ 2017-05-17 21:15 UTC (permalink / raw)
To: buildroot
Hello All!
This small series aims at fixing the build of qt-5.8 on sparc.
Qt 5.8 needs atomic operations, which on some architectures are compiler
builtins, while on other archs they are provided by a helper library,
libatomic. For sparc, atomics are in libatomic.
First, we introduce a new qmake config snippet, arch.conf, which is
included from our qmake.conf.
By default, that file is empty, except when libatomic is present in the
toolchain, in which case it add -latomic to the list of libraries to
link with.
Regards,
Yann E. MORIN.
The following changes since commit 19e78e509e5e729437d2188b28aee7655340c67d
oracle-mysql: add patch to fix gcc 6.x build issue (2017-05-17 22:24:14 +0200)
are available in the git repository at:
git://git.buildroot.org/~ymorin/git/buildroot.git
for you to fetch changes up to 6bdfcdff79478a9835c4c5d0296f8f62d5b64d66
package/qt5base: link with -latomic for sparc (2017-05-17 23:08:55 +0200)
----------------------------------------------------------------
Yann E. MORIN (2):
package/qt5base: install arch-specific configuration
package/qt5base: link with -latomic for sparc
package/qt5/qt5base/qmake.conf | 3 +++
package/qt5/qt5base/qt5base.mk | 10 ++++++++++
2 files changed, 13 insertions(+)
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 1/2 v2] package/qt5base: install arch-specific configuration
2017-05-17 21:15 [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation Yann E. MORIN
@ 2017-05-17 21:15 ` Yann E. MORIN
2017-05-27 12:41 ` Bernd Kuhls
2017-05-17 21:15 ` [Buildroot] [PATCH 2/2 v2] package/qt5base: link with -latomic for sparc Yann E. MORIN
2017-06-08 21:21 ` [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation Thomas Petazzoni
2 siblings, 1 reply; 8+ messages in thread
From: Yann E. MORIN @ 2017-05-17 21:15 UTC (permalink / raw)
To: buildroot
For some architectures, it is necessary to link with additional
libraries, such as -latomic for sparc.
Add a bit of support to make that easy; actual use will be introduced in
the following patch.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Julien Corjon <corjon.j@ecagroup.com>
Cc: Peter Seiderer <ps.report@gmx.net>
---
Changes v1 -> v2:
- rename macro
---
package/qt5/qt5base/qmake.conf | 3 +++
package/qt5/qt5base/qt5base.mk | 3 +++
2 files changed, 6 insertions(+)
diff --git a/package/qt5/qt5base/qmake.conf b/package/qt5/qt5base/qmake.conf
index 49cf898316..8b6debe583 100644
--- a/package/qt5/qt5base/qmake.conf
+++ b/package/qt5/qt5base/qmake.conf
@@ -21,5 +21,8 @@ CONFIG += nostrip
QMAKE_LIBS += -lrt -lpthread -ldl
QMAKE_CFLAGS_ISYSTEM =
+# Architecturespecific configuration
+include(arch.conf)
+
include(../common/linux_device_post.conf)
load(qt_config)
diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
index 0c088350e5..81774b74a6 100644
--- a/package/qt5/qt5base/qt5base.mk
+++ b/package/qt5/qt5base/qt5base.mk
@@ -214,12 +214,15 @@ define QT5BASE_CONFIGURE_CONFIG_FILE
endef
endif
+QT5BASE_ARCH_CONFIG_FILE = $(@D)/mkspecs/devices/linux-buildroot-g++/arch.conf
define QT5BASE_CONFIGURE_CMDS
$(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qmake.conf \
$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
$(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qplatformdefs.h \
$(@D)/mkspecs/devices/linux-buildroot-g++/qplatformdefs.h
$(QT5BASE_CONFIGURE_CONFIG_FILE)
+ touch $(QT5BASE_ARCH_CONFIG_FILE)
+ $(QT5BASE_CONFIGURE_ARCH_CONFIG)
(cd $(@D); \
$(TARGET_MAKE_ENV) \
PKG_CONFIG="$(PKG_CONFIG_HOST_BINARY)" \
--
2.11.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 2/2 v2] package/qt5base: link with -latomic for sparc
2017-05-17 21:15 [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation Yann E. MORIN
2017-05-17 21:15 ` [Buildroot] [PATCH 1/2 v2] package/qt5base: install arch-specific configuration Yann E. MORIN
@ 2017-05-17 21:15 ` Yann E. MORIN
2017-05-18 3:34 ` Baruch Siach
2017-05-27 12:41 ` Bernd Kuhls
2017-06-08 21:21 ` [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation Thomas Petazzoni
2 siblings, 2 replies; 8+ messages in thread
From: Yann E. MORIN @ 2017-05-17 21:15 UTC (permalink / raw)
To: buildroot
Qt 5.8 needs atomic operations, which are provided in libatomic when it
exists, ike on sparc with gcc >= 4.8.
Fixes:
http://autobuild.buildroot.org/results/49b/49bc9345b9849c9c3c53ace290c534ff7bb98683/
http://autobuild.buildroot.org/results/9f2/9f213406954be51dfcad76ebdce8b73850842180/
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Julien Corjon <corjon.j@ecagroup.com>
Cc: Peter Seiderer <ps.report@gmx.net>
---
Changes v1 -> v2:
- condition is on HAS_LIBATOMIC, not on BR2_sparc (Thomas)
---
package/qt5/qt5base/qt5base.mk | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
index 81774b74a6..5944c869bb 100644
--- a/package/qt5/qt5base/qt5base.mk
+++ b/package/qt5/qt5base/qt5base.mk
@@ -215,6 +215,13 @@ endef
endif
QT5BASE_ARCH_CONFIG_FILE = $(@D)/mkspecs/devices/linux-buildroot-g++/arch.conf
+ifeq ($(BR2_TOOLCHAIN_HAS_LIBATOMIC)$(BR2_PACKAGE_QT5_VERSION_LATEST),yy)
+# Qt 5.8 needs atomics, which on sparc are in -latomic
+define QT5BASE_CONFIGURE_ARCH_CONFIG
+ printf 'LIBS += -latomic\n' >$(QT5BASE_ARCH_CONFIG_FILE)
+endef
+endif
+
define QT5BASE_CONFIGURE_CMDS
$(INSTALL) -m 0644 -D $(QT5BASE_PKGDIR)/qmake.conf \
$(@D)/mkspecs/devices/linux-buildroot-g++/qmake.conf
--
2.11.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 2/2 v2] package/qt5base: link with -latomic for sparc
2017-05-17 21:15 ` [Buildroot] [PATCH 2/2 v2] package/qt5base: link with -latomic for sparc Yann E. MORIN
@ 2017-05-18 3:34 ` Baruch Siach
2017-05-27 12:41 ` Bernd Kuhls
1 sibling, 0 replies; 8+ messages in thread
From: Baruch Siach @ 2017-05-18 3:34 UTC (permalink / raw)
To: buildroot
Hi Yann,
On Wed, May 17, 2017 at 11:15:29PM +0200, Yann E. MORIN wrote:
> Qt 5.8 needs atomic operations, which are provided in libatomic when it
> exists, ike on sparc with gcc >= 4.8.
>
> Fixes:
> http://autobuild.buildroot.org/results/49b/49bc9345b9849c9c3c53ace290c534ff7bb98683/
> http://autobuild.buildroot.org/results/9f2/9f213406954be51dfcad76ebdce8b73850842180/
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Julien Corjon <corjon.j@ecagroup.com>
> Cc: Peter Seiderer <ps.report@gmx.net>
>
> ---
> Changes v1 -> v2:
> - condition is on HAS_LIBATOMIC, not on BR2_sparc (Thomas)
The commit subject line should also to match that change.
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 2/2 v2] package/qt5base: link with -latomic for sparc
2017-05-17 21:15 ` [Buildroot] [PATCH 2/2 v2] package/qt5base: link with -latomic for sparc Yann E. MORIN
2017-05-18 3:34 ` Baruch Siach
@ 2017-05-27 12:41 ` Bernd Kuhls
1 sibling, 0 replies; 8+ messages in thread
From: Bernd Kuhls @ 2017-05-27 12:41 UTC (permalink / raw)
To: buildroot
Am Wed, 17 May 2017 23:15:29 +0200 schrieb Yann E. MORIN:
> Qt 5.8 needs atomic operations, which are provided in libatomic when it
> exists, ike on sparc with gcc >= 4.8.
>
> Fixes:
> http://autobuild.buildroot.org/
results/49b/49bc9345b9849c9c3c53ace290c534ff7bb98683/
> http://autobuild.buildroot.org/
results/9f2/9f213406954be51dfcad76ebdce8b73850842180/
>
> Signed-off-by: "Yann E. MORIN"
> <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni
> <thomas.petazzoni@free-electrons.com>
> Cc: Julien Corjon <corjon.j@ecagroup.com>
> Cc: Peter Seiderer <ps.report@gmx.net>
Tested-by: Bernd Kuhls <bernd.kuhls@t-online.de>
(compile-tested using http://autobuild.buildroot.net/
results/967/967e7b7b6960ff49afc5c845832abd49d93ed676/ )
This patch series also fixes http://autobuild.buildroot.net/
results/967/967e7b7b6960ff49afc5c845832abd49d93ed676/
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 1/2 v2] package/qt5base: install arch-specific configuration
2017-05-17 21:15 ` [Buildroot] [PATCH 1/2 v2] package/qt5base: install arch-specific configuration Yann E. MORIN
@ 2017-05-27 12:41 ` Bernd Kuhls
0 siblings, 0 replies; 8+ messages in thread
From: Bernd Kuhls @ 2017-05-27 12:41 UTC (permalink / raw)
To: buildroot
Am Wed, 17 May 2017 23:15:28 +0200 schrieb Yann E. MORIN:
> For some architectures, it is necessary to link with additional
> libraries, such as -latomic for sparc.
>
> Add a bit of support to make that easy; actual use will be introduced in
> the following patch.
>
> Signed-off-by: "Yann E. MORIN"
> <yann.morin.1998@free.fr> Cc: Julien Corjon
> <corjon.j@ecagroup.com>
> Cc: Peter Seiderer <ps.report@gmx.net>
Tested-by: Bernd Kuhls <bernd.kuhls@t-online.de>
(compile-tested using http://autobuild.buildroot.net/
results/967/967e7b7b6960ff49afc5c845832abd49d93ed676/ )
This patch series also fixes http://autobuild.buildroot.net/
results/967/967e7b7b6960ff49afc5c845832abd49d93ed676/
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation
2017-05-17 21:15 [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation Yann E. MORIN
2017-05-17 21:15 ` [Buildroot] [PATCH 1/2 v2] package/qt5base: install arch-specific configuration Yann E. MORIN
2017-05-17 21:15 ` [Buildroot] [PATCH 2/2 v2] package/qt5base: link with -latomic for sparc Yann E. MORIN
@ 2017-06-08 21:21 ` Thomas Petazzoni
2017-06-09 11:21 ` Peter Korsgaard
2 siblings, 1 reply; 8+ messages in thread
From: Thomas Petazzoni @ 2017-06-08 21:21 UTC (permalink / raw)
To: buildroot
Hello,
On Wed, 17 May 2017 23:15:23 +0200, Yann E. MORIN wrote:
> Yann E. MORIN (2):
> package/qt5base: install arch-specific configuration
> package/qt5base: link with -latomic for sparc
Both applied to master. Thanks!
Peter: those are fixing a build issue, but only on Sparc, so I don't
know how much we want those patches in the LTS branch.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation
2017-06-08 21:21 ` [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation Thomas Petazzoni
@ 2017-06-09 11:21 ` Peter Korsgaard
0 siblings, 0 replies; 8+ messages in thread
From: Peter Korsgaard @ 2017-06-09 11:21 UTC (permalink / raw)
To: buildroot
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:
> Hello,
> On Wed, 17 May 2017 23:15:23 +0200, Yann E. MORIN wrote:
>> Yann E. MORIN (2):
>> package/qt5base: install arch-specific configuration
>> package/qt5base: link with -latomic for sparc
> Both applied to master. Thanks!
> Peter: those are fixing a build issue, but only on Sparc, so I don't
> know how much we want those patches in the LTS branch.
They seem quite safe and a test build with them applied on the LTS
branch succeeded, so committed to 2017.02.x and 2017.05.x, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-06-09 11:21 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-17 21:15 [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation Yann E. MORIN
2017-05-17 21:15 ` [Buildroot] [PATCH 1/2 v2] package/qt5base: install arch-specific configuration Yann E. MORIN
2017-05-27 12:41 ` Bernd Kuhls
2017-05-17 21:15 ` [Buildroot] [PATCH 2/2 v2] package/qt5base: link with -latomic for sparc Yann E. MORIN
2017-05-18 3:34 ` Baruch Siach
2017-05-27 12:41 ` Bernd Kuhls
2017-06-08 21:21 ` [Buildroot] [PATCH 0/2 v2] package/qt5base: fix sparc compilation Thomas Petazzoni
2017-06-09 11:21 ` Peter Korsgaard
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.