All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.