* [Buildroot] [PATCH v1] package/qt5base: disable feature-relocatable
@ 2020-05-17 10:28 Peter Seiderer
2020-05-17 19:26 ` Yann E. MORIN
2020-05-29 21:24 ` Peter Korsgaard
0 siblings, 2 replies; 4+ messages in thread
From: Peter Seiderer @ 2020-05-17 10:28 UTC (permalink / raw)
To: buildroot
Disable feature-relocatable to avoid a path mismatch while searching qml
files and buildroot BR2_ROOTFS_MERGED_USR feature enabled.
As described in [1] with feature-relocatable Qt determines the search
pathes relative to the location of libQt5Core.so, with BR2_ROOTFS_MERGED_USR
enabled found first under the symlink path '/lib' instead of the install
path '/usr/lib' and searches sequentially for qml files under '/qml' instead
of the correct '/usr/qml'.
Fixes:
- https://bugs.busybox.net/show_bug.cgi?id=12906
[1] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=4ac872639ed0dd3ae6627e05bdda821f7d128500
Reported-by: Joonas Harjum?ki <jharjuma@gmail.com>
Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
package/qt5/qt5base/qt5base.mk | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
index 97315496b9..374285a825 100644
--- a/package/qt5/qt5base/qt5base.mk
+++ b/package/qt5/qt5base/qt5base.mk
@@ -18,6 +18,9 @@ QT5BASE_INSTALL_STAGING = YES
# want to use the Buildroot packaged zlib
# * -system-pcre because pcre is mandatory to build Qt, and we
# want to use the one packaged in Buildroot
+# * -no-feature-relocatable to work around path mismatch
+# while searching qml files and buildroot BR2_ROOTFS_MERGED_USR
+# feature enabled
QT5BASE_CONFIGURE_OPTS += \
-optimized-qmake \
-no-cups \
@@ -25,7 +28,8 @@ QT5BASE_CONFIGURE_OPTS += \
-system-zlib \
-system-pcre \
-no-pch \
- -shared
+ -shared \
+ -no-feature-relocatable
# starting from version 5.9.0, -optimize-debug is enabled by default
# for debug builds and it overrides -O* with -Og which is not what we
--
2.26.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH v1] package/qt5base: disable feature-relocatable
2020-05-17 10:28 [Buildroot] [PATCH v1] package/qt5base: disable feature-relocatable Peter Seiderer
@ 2020-05-17 19:26 ` Yann E. MORIN
2020-05-29 21:24 ` Peter Korsgaard
1 sibling, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2020-05-17 19:26 UTC (permalink / raw)
To: buildroot
Peter, All,
On 2020-05-17 12:28 +0200, Peter Seiderer spake thusly:
> Disable feature-relocatable to avoid a path mismatch while searching qml
> files and buildroot BR2_ROOTFS_MERGED_USR feature enabled.
>
> As described in [1] with feature-relocatable Qt determines the search
> pathes relative to the location of libQt5Core.so, with BR2_ROOTFS_MERGED_USR
> enabled found first under the symlink path '/lib' instead of the install
> path '/usr/lib' and searches sequentially for qml files under '/qml' instead
> of the correct '/usr/qml'.
>
> Fixes:
> - https://bugs.busybox.net/show_bug.cgi?id=12906
>
> [1] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=4ac872639ed0dd3ae6627e05bdda821f7d128500
>
> Reported-by: Joonas Harjum?ki <jharjuma@gmail.com>
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
Applied to master, thanks. See a minor nit, below...
> ---
> package/qt5/qt5base/qt5base.mk | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/package/qt5/qt5base/qt5base.mk b/package/qt5/qt5base/qt5base.mk
> index 97315496b9..374285a825 100644
> --- a/package/qt5/qt5base/qt5base.mk
> +++ b/package/qt5/qt5base/qt5base.mk
> @@ -18,6 +18,9 @@ QT5BASE_INSTALL_STAGING = YES
> # want to use the Buildroot packaged zlib
> # * -system-pcre because pcre is mandatory to build Qt, and we
> # want to use the one packaged in Buildroot
> +# * -no-feature-relocatable to work around path mismatch
> +# while searching qml files and buildroot BR2_ROOTFS_MERGED_USR
> +# feature enabled
$ make check-package
package/qt5/qt5base/qt5base.mk:23: line contains trailing whitespace
Fixed, and applied to master.
Regards,
Yann E. MORIN.
> QT5BASE_CONFIGURE_OPTS += \
> -optimized-qmake \
> -no-cups \
> @@ -25,7 +28,8 @@ QT5BASE_CONFIGURE_OPTS += \
> -system-zlib \
> -system-pcre \
> -no-pch \
> - -shared
> + -shared \
> + -no-feature-relocatable
>
> # starting from version 5.9.0, -optimize-debug is enabled by default
> # for debug builds and it overrides -O* with -Og which is not what we
> --
> 2.26.2
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH v1] package/qt5base: disable feature-relocatable
2020-05-17 10:28 [Buildroot] [PATCH v1] package/qt5base: disable feature-relocatable Peter Seiderer
2020-05-17 19:26 ` Yann E. MORIN
@ 2020-05-29 21:24 ` Peter Korsgaard
2020-05-29 21:56 ` Peter Seiderer
1 sibling, 1 reply; 4+ messages in thread
From: Peter Korsgaard @ 2020-05-29 21:24 UTC (permalink / raw)
To: buildroot
>>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:
> Disable feature-relocatable to avoid a path mismatch while searching qml
> files and buildroot BR2_ROOTFS_MERGED_USR feature enabled.
> As described in [1] with feature-relocatable Qt determines the search
> pathes relative to the location of libQt5Core.so, with BR2_ROOTFS_MERGED_USR
> enabled found first under the symlink path '/lib' instead of the install
> path '/usr/lib' and searches sequentially for qml files under '/qml' instead
> of the correct '/usr/qml'.
> Fixes:
> - https://bugs.busybox.net/show_bug.cgi?id=12906
> [1] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=4ac872639ed0dd3ae6627e05bdda821f7d128500
> Reported-by: Joonas Harjum?ki <jharjuma@gmail.com>
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
This is only for Qt 5.14 (E.G. master and not 2020.02.x), right? From a
quick look I don't see this option in the 5.12.8 version we have on the
LTS branch.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH v1] package/qt5base: disable feature-relocatable
2020-05-29 21:24 ` Peter Korsgaard
@ 2020-05-29 21:56 ` Peter Seiderer
0 siblings, 0 replies; 4+ messages in thread
From: Peter Seiderer @ 2020-05-29 21:56 UTC (permalink / raw)
To: buildroot
Hello Peter,
On Fri, 29 May 2020 23:24:38 +0200, Peter Korsgaard <peter@korsgaard.com> wrote:
> >>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:
>
> > Disable feature-relocatable to avoid a path mismatch while searching qml
> > files and buildroot BR2_ROOTFS_MERGED_USR feature enabled.
>
> > As described in [1] with feature-relocatable Qt determines the search
> > pathes relative to the location of libQt5Core.so, with BR2_ROOTFS_MERGED_USR
> > enabled found first under the symlink path '/lib' instead of the install
> > path '/usr/lib' and searches sequentially for qml files under '/qml' instead
> > of the correct '/usr/qml'.
>
> > Fixes:
> > - https://bugs.busybox.net/show_bug.cgi?id=12906
>
> > [1] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=4ac872639ed0dd3ae6627e05bdda821f7d128500
>
> > Reported-by: Joonas Harjum?ki <jharjuma@gmail.com>
> > Signed-off-by: Peter Seiderer <ps.report@gmx.net>
>
> This is only for Qt 5.14 (E.G. master and not 2020.02.x), right? From a
> quick look I don't see this option in the 5.12.8 version we have on the
> LTS branch.
>
Yes...
Regards,
Peter
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-05-29 21:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-17 10:28 [Buildroot] [PATCH v1] package/qt5base: disable feature-relocatable Peter Seiderer
2020-05-17 19:26 ` Yann E. MORIN
2020-05-29 21:24 ` Peter Korsgaard
2020-05-29 21:56 ` Peter Seiderer
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.