Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Zoltan Gyarmati <mr.zoltan.gyarmati@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v6 8/9] python-pyqt5: Depend directly on Qt5 rather than its toolchain requirements
Date: Thu, 2 Feb 2017 09:57:57 +0100	[thread overview]
Message-ID: <38d15b3f-1032-4cda-b73e-9489c8a11806@gmail.com> (raw)
In-Reply-To: <20170201222915.58e4f805@free-electrons.com>

On 02/01/2017 10:29 PM, Thomas Petazzoni wrote:

> Hello,
>
> On Tue, 31 Jan 2017 08:59:10 +0100, Zoltan Gyarmati wrote:
>> From: Naumann Andreas <ANaumann@ultratronik.de>
>>
>>  When pyqt5 is used it's obvious that qt5 needs to be selected and configured
>> by the user, hence we enforce it by making pyqt5 depending on qt5, rather than
>> selecting it and depending only on the qt5 requirements.
>>
>> Signed-off-by: Andreas Naumann <anaumann@ultratronik.de>
>> Signed-off-by: Zoltan Gyarmati <mr.zoltan.gyarmati@gmail.com>
> The previous version of this patch series from Andreas was doing this
> change for both python-pyqt and python-pyqt5. But in your new version,
> you have removed Andreas patch making the change to python-pyqt, and
> only kept the python-pyqt5 change. Why? Due to this, the handling of
> python-pyqt would no longer be consistent with the one of python-pyqt5.

Ah, i was focusing on the qt5 related things, but you are right, it
should be consistent, i'll add it back, thanks for noting this. Did you
find any other flaw in the series?
>
> Thanks,
>
> Thomas
Thanks,

Zoltan Gyarmati
https://zgyarmati.de


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20170202/84cc8b61/attachment.asc>

  reply	other threads:[~2017-02-02  8:57 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-31  7:59 [Buildroot] [PATCH v6 0/9] bump/choice for qt 5.8.0 Zoltan Gyarmati
2017-01-31  7:59 ` [Buildroot] [PATCH v6 1/9] qt5: add choice between version 5.6.2 and 5.8.0 Zoltan Gyarmati
2017-02-02 19:31   ` Peter Seiderer
2017-02-06 13:32     ` Arnout Vandecappelle
2017-02-02 22:01   ` Arnout Vandecappelle
2017-01-31  7:59 ` [Buildroot] [PATCH v6 2/9] qt5enginio : move into legacy compatibility Zoltan Gyarmati
2017-02-06 13:51   ` Arnout Vandecappelle
2017-01-31  7:59 ` [Buildroot] [PATCH v6 3/9] qt53d/qt5quickcontrols2/qt5serialbus : move out of tech preview Zoltan Gyarmati
2017-01-31  7:59 ` [Buildroot] [PATCH v6 4/9] Conditional revert "qt5base: install bundled fonts to target" Zoltan Gyarmati
2017-01-31  7:59 ` [Buildroot] [PATCH v6 5/9] qt5declarative: enable Quick for non-GL platforms for Qt 5.8.0 Zoltan Gyarmati
2017-02-02 22:20   ` Arnout Vandecappelle
2017-02-07  8:41     ` Andreas Naumann
2017-02-07  9:39       ` Zoltan Gyarmati
2017-02-07  9:54         ` Andreas Naumann
2017-01-31  7:59 ` [Buildroot] [PATCH v6 6/9] qt5quickcontrols: More deterministic target install Zoltan Gyarmati
2017-01-31  7:59 ` [Buildroot] [PATCH v6 7/9] qt5quickcontrols/qt5quickcontrols2: enable for software renderer for Qt 5.8.0 Zoltan Gyarmati
2017-02-01 20:18   ` Peter Seiderer
2017-02-02  8:55     ` Zoltan Gyarmati
2017-02-02 18:43       ` Peter Seiderer
2017-01-31  7:59 ` [Buildroot] [PATCH v6 8/9] python-pyqt5: Depend directly on Qt5 rather than its toolchain requirements Zoltan Gyarmati
2017-02-01 21:29   ` Thomas Petazzoni
2017-02-02  8:57     ` Zoltan Gyarmati [this message]
2017-02-02 11:00       ` Thomas Petazzoni
2017-02-02 23:06   ` Arnout Vandecappelle
2017-02-07  8:22     ` Andreas Naumann
2017-01-31  7:59 ` [Buildroot] [PATCH v6 9/9] package/python-pyqt5: add patch to fix build with Qt 5.8.0 Zoltan Gyarmati
2017-02-02 23:15   ` Arnout Vandecappelle
2017-02-06  8:57     ` Zoltan Gyarmati
2017-02-06 13:28       ` Arnout Vandecappelle
2017-02-06 15:40 ` [Buildroot] [PATCH v6 0/9] bump/choice for qt 5.8.0 Thomas Petazzoni
2017-02-06 17:44   ` Andreas Naumann
2017-02-06 20:24     ` Arnout Vandecappelle
2017-02-06 20:36       ` Peter Seiderer
2017-02-06 20:54         ` Arnout Vandecappelle
2017-02-06 21:06           ` Peter Seiderer
2017-02-06 21:09             ` Arnout Vandecappelle
2017-02-06 21:27               ` Peter Seiderer
2017-02-07 17:08 ` Thomas Petazzoni
2017-02-07 17:21   ` Zoltan Gyarmati

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=38d15b3f-1032-4cda-b73e-9489c8a11806@gmail.com \
    --to=mr.zoltan.gyarmati@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox