From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 2/2] python-pyqt5: new package
Date: Fri, 19 Aug 2016 16:48:00 +0200 [thread overview]
Message-ID: <20160819164800.68305618@free-electrons.com> (raw)
In-Reply-To: <4a764858506ee90fcb41d57d252e5864@manuel-voegele.de>
Hello,
On Fri, 19 Aug 2016 16:40:05 +0200, Manuel V?gele wrote:
> > Since I guess PyQt5 is managed in Git upstream, I would very much
> > prefer a Git formatted patch if possible. Of course, if Git is not used
> > upstream, then it's fine.
>
> I didn't manage to find any official PyQt repository. But if it exists I
> suppose it would be a Mercurial repo since all the other Riverbank
> projects use Mercurial too.
OK. Then the patch format is OK as-is.
> >> +# Since we can't run generate qtdetail.out by running qtdetail on
> >> target device
> >> +# we must generate the configuration.
> >> +define PYTHON_PYQT5_GENERATE_QTDETAIL
> >> + $(RM) -f $(1)/qtdetail.out
> >> + $(call PYTHON_PYQT5_QTDETAIL,$(PYTHON_PYQT5_QTDETAIL_LICENSE),$(1))
> >> + $(call PYTHON_PYQT5_QTDETAIL,$(PYTHON_PYQT5_QTDETAIL_TYPE),$(1))
> >> + $(foreach f,$(PYTHON_PYQT5_QTDETAIL_DISABLE_FEATURES),
> >> + $(call PYTHON_PYQT5_QTDETAIL,$(f),$(1)) \
> >> + )
> >> +endef
> >
> > So this is all exactly the same as PyQt4 ?
> It's not all the same - I already removed some switches which aren't
> required in PyQt5. But I'm gonna recheck if I can remove even more.
OK.
> > Can you test after removing this compileall line? Indeed, we are now
> > byte-compiling all Python modules globally. If it works, a patch
> > removing this line for python-pyqt.mk would be good to have.
>
> I'm gonna do that.
Great, thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
next prev parent reply other threads:[~2016-08-19 14:48 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-19 13:42 [Buildroot] [PATCH 0/2] PyQt5 package Manuel Vögele
2016-08-19 13:42 ` [Buildroot] [PATCH 1/2] python-sip: allow package to be built using Qt5 Manuel Vögele
2016-08-19 13:58 ` Thomas Petazzoni
2016-08-19 13:42 ` [Buildroot] [PATCH 2/2] python-pyqt5: new package Manuel Vögele
2016-08-19 14:07 ` Thomas Petazzoni
2016-08-19 14:40 ` Manuel Vögele
2016-08-19 14:48 ` Thomas Petazzoni [this message]
2016-08-23 9:26 ` [Buildroot] [PATCH v2] " Manuel Vögele
2016-09-20 19:14 ` Thomas Petazzoni
2016-09-22 20:54 ` Arnout Vandecappelle
2016-09-27 14:45 ` Thomas Petazzoni
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=20160819164800.68305618@free-electrons.com \
--to=thomas.petazzoni@free-electrons.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