From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 0/4] Bump to Python 3.5 and pyc compilation improvements
Date: Sun, 1 May 2016 21:29:33 +0200 [thread overview]
Message-ID: <20160501212933.4c423ee5@free-electrons.com> (raw)
In-Reply-To: <CAHXCMMJ+01WoVug=55dYer2175Wsvdvfo4GTBfTQhi6j-2Ge+w@mail.gmail.com>
Hello,
On Sun, 1 May 2016 15:51:30 +0200, Samuel Martin wrote:
> > From your feedback, I believe patches 1 and 2 could be applied, but not
> > patch 3 and 4. Is this correct?
> Exactly.
Ok, good. In fact, I will shortly resend a patch series with the
additional fixes.
> > Or should we simply say that the packages that install Python stuff
> > outside of usr/lib/pythonX.Y are responsible for doing their own stuff?
> I tend to prefer this second approach.
> Doing things can have undesired side-effects.
Agreed.
> > What do you suggest to handle this one? It seems to be an intentional
> > failure.
> Yes it is intentional. Removing this file (that contains nothing but a
> syntax error...) in post-install hook is ok.
I've fixed it by adding a patch that removes the syntax error, and I've
reported the problem upstream:
https://github.com/crossbario/crossbar/issues/750.
> >> - python-pyftpdlib [4]
> >
> > This one is clearly a python 3 compatibility problem, which is trivial
> > to fix. But the package is said by upstream to support Python 3, so
> > it's a bit weird.
I've fixed that one as well. However, one weird thing is that the
offending file (splice.py) is in the tarball provided on pypi.org, but
not in the project official Github repository, in its 1.5.0 tag.
I am not sure where this splice.py is coming from...
> >> - python-pygame [5,6]
> >
> > Same here, those are trivial Python 3 issues. Why do we mark the
> > package as available for Python 3 if it doesn't actually build ? :-)
This one has been fixed upstream, by
https://bitbucket.org/pygame/pygame/diff/examples/macosx/aliens_app_example/aliens.py?diff2=22efec0b44da&at=default,
I will either backport the patch, or bump the package version.
> > Is it broken install rules, or something that's purely intentional?
> > Also, note that those are not Python modules, there is no __init__.py
> > in those directories. Which probably means they would anyway not be
> > compiled as .pyc anyway.
> This package needs further investigations, I did not notice the same
> behavior with python2.
OK. But since it's outside usr/lib/pythonX.Y, I don't think we care too
much at this point.
Thanks for your feedback!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2016-05-01 19:29 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-30 12:47 [Buildroot] [PATCH v2 0/4] Bump to Python 3.5 and pyc compilation improvements Thomas Petazzoni
2016-04-30 12:47 ` [Buildroot] [PATCH v2 1/4] python3: bump to 3.5.1 Thomas Petazzoni
2016-04-30 20:56 ` Samuel Martin
2016-04-30 12:47 ` [Buildroot] [PATCH v2 2/4] python: align the target finalize hooks with python3 Thomas Petazzoni
2016-04-30 20:56 ` Samuel Martin
2016-05-01 7:36 ` Thomas Petazzoni
2016-05-01 7:38 ` Samuel Martin
2016-05-01 13:08 ` Thomas Petazzoni
2016-04-30 12:47 ` [Buildroot] [PATCH v2 3/4] python/python3: globalize *.pyc files compilation Thomas Petazzoni
2016-04-30 20:57 ` Samuel Martin
2016-04-30 12:47 ` [Buildroot] [PATCH v2 4/4] samba4: remove compilation of .pyc files Thomas Petazzoni
2016-04-30 20:57 ` Samuel Martin
2016-04-30 20:56 ` [Buildroot] [PATCH v2 0/4] Bump to Python 3.5 and pyc compilation improvements Samuel Martin
2016-05-01 13:26 ` Thomas Petazzoni
2016-05-01 13:51 ` Samuel Martin
2016-05-01 14:14 ` Yegor Yefremov
2016-05-01 17:11 ` Thomas Petazzoni
2016-05-01 17:21 ` Yegor Yefremov
2016-05-01 19:53 ` Thomas Petazzoni
2016-05-01 19:29 ` Thomas Petazzoni [this message]
2016-05-01 20:17 ` 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=20160501212933.4c423ee5@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