From: jeremie.galarneau@efficios.com (Jérémie Galarneau)
Subject: [lttng-dev] [PATCH babeltrace 1/2] Add missing shebangs to Python examples.
Date: Fri, 25 Jan 2013 16:03:12 +0000 [thread overview]
Message-ID: <CA+jJMxuo=r+k+yPdNneE_W9J86pwRgOF9R9=mo8QfMwZO4peOw@mail.gmail.com> (raw)
In-Reply-To: <CA+jJMxvqEXqgK8oMmJZT5WKiKuNYm7Wws47QOi=e2wzYnaKH8A@mail.gmail.com>
Looks like this patch will require more work since it was decided we
only want to explicitly support Python3. I'll have to look into the
examples and see what can be ported and fixed. I'll also work on
having a more comprehensive test suite in preparation for an eventual
merge into master.
However, please note that the second patch from this series - [PATCH
babeltrace 2/2] Fix: Unified types expected by the Definition Python
interface. - will remain unchanged and should be pulled as it fixes a
bug that prevented the Python iterator tests from passing.
Regards,
J?r?mie
On Wed, Jan 23, 2013 at 5:21 PM, J?r?mie Galarneau
<jeremie.galarneau at gmail.com> wrote:
>> I would recommend using "#!/usr/bin/env python" so it will default to
>> the Python interpreter available on the machine. Take note that we
>> might have issues with Python 3 interpreters if the scripts are not
>> fully compliant.
>
> It does seem like some of the example scripts are not Python 3
> compatible. However, forcing the use of Python 2 on distros wich use
> Python 3 as the default interpreter is far from ideal... I'll see if I
> can make them version agnostic before resubmitting.
>
> Thanks,
> J?r?mie
prev parent reply other threads:[~2013-01-25 16:03 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-22 23:47 Jérémie Galarneau
2013-01-22 23:47 ` [lttng-dev] [PATCH babeltrace 2/2] Fix: Unified types expected by the Definition Python interface Jérémie Galarneau
2013-01-25 16:39 ` Mathieu Desnoyers
2013-01-23 14:45 ` [lttng-dev] [PATCH babeltrace 1/2] Add missing shebangs to Python examples Mathieu Desnoyers
2013-01-23 17:05 ` Christian Babeux
[not found] ` <CA+jJMxvqEXqgK8oMmJZT5WKiKuNYm7Wws47QOi=e2wzYnaKH8A@mail.gmail.com>
2013-01-25 16:03 ` Jérémie Galarneau [this message]
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='CA+jJMxuo=r+k+yPdNneE_W9J86pwRgOF9R9=mo8QfMwZO4peOw@mail.gmail.com' \
--to=jeremie.galarneau@efficios.com \
/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