From: Joshua Henderson <joshua.henderson@microchip.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v3] package/qt5location: handle plugins being conditionally built
Date: Fri, 28 Jul 2017 09:31:13 -0700 [thread overview]
Message-ID: <49674504-df74-e429-b49f-eacf299ef15b@microchip.com> (raw)
In-Reply-To: <20170725215220.6d447778@windsurf.home>
Thomas, Peter,
On 07/25/2017 12:52 PM, Thomas Petazzoni wrote:
> Hello,
>
> On Mon, 24 Jul 2017 12:50:03 -0700, Joshua Henderson wrote:
>> Fixes a build issue [1] where the qt5location plugins are not built, but are
>> attempted to be installed by buildroot.
>>
>> The qt5location plugins have dependencies that are automatically and incorrectly
>> resolved causing plugins to conditionally be built based on whether quick is
>> available. This change adds a patch for an upstream bug [2] that builds the
>> plugins directory irrelevant of the quick module.
>>
>> [1] http://autobuild.buildroot.net/results/bc1/bc13abf3bb2fe1c991aec2334ee658c9641d1fd5/build-end.log
>> [2] https://bugreports.qt.io/browse/QTBUG-62098
>>
>> Cc: Julien Corjon <corjon.j@ecagroup.com>
>> Cc: Peter Seiderer <ps.report@gmx.net>
>> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
>> Signed-off-by: Joshua Henderson <joshua.henderson@microchip.com>
>>
>> --
>>
>> Changes for v3:
>> - Change commit message to be more inline with actual fix.
>> - Issue with upstream patch - use Peter's upstream patch instead.
>>
>> Changes for v2:
>> - Instead of skipping missing files, fix the root cause of improper dependency
>> on quick when building the position plugin.
>> ---
>> ...001-Fix-plugins-build-dependency-tracking.patch | 39 ++++++++++++++++++++++
>> 1 file changed, 39 insertions(+)
>> create mode 100644 package/qt5/qt5location/5.9.1/0001-Fix-plugins-build-dependency-tracking.patch
>
> Thanks, but Peter had sent an almost identical patch a few hours
> before, and I've applied his patch.
It wasn't exactly identical. The applied patch now causes a build failure for 5.6.2:
qt5location 5.6.2 Patching^[
Applying 0001-Fix-plugins-build-dependency.patch using patch:
patching file src/src.pro
Hunk #1 FAILED at 7.
Hunk #2 succeeded at 14 (offset -4 lines).
1 out of 2 hunks FAILED -- saving rejects to file src/src.pro.rej
>
> Thanks a lot for all this work on Qt5, much appreciated! Note that we
> still have a build failure of qt5webkit 5.6.1:
>
> http://autobuild.buildroot.net/results/2f3/2f381967c82775ad911a246a9607034ef489c1ce/build-end.log
>
I also have several other patches to rev (add qt5wayland and license hashes for all of qt5).
Josh
next prev parent reply other threads:[~2017-07-28 16:31 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-24 19:50 [Buildroot] [PATCH v3] package/qt5location: handle plugins being conditionally built Joshua Henderson
2017-07-25 19:52 ` Thomas Petazzoni
2017-07-27 7:09 ` Peter Seiderer
2017-07-28 16:31 ` Joshua Henderson [this message]
2017-07-28 21:17 ` Arnout Vandecappelle
2017-07-28 21:54 ` Joshua Henderson
2017-07-28 21:59 ` Thomas Petazzoni
2017-07-28 22:08 ` Peter Seiderer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=49674504-df74-e429-b49f-eacf299ef15b@microchip.com \
--to=joshua.henderson@microchip.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.