All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] package: add python-raven
Date: Fri, 3 Nov 2017 21:00:29 +0100	[thread overview]
Message-ID: <20171103210029.6ddf4546@windsurf> (raw)
In-Reply-To: <CAMWSM7gV0Fg64+fbGMF9P00N9mnmu3VEugiYKc=S0sQpO6aeDA@mail.gmail.com>

Hello,

On Fri, 3 Nov 2017 12:08:43 -0700, Joseph Kogut wrote:

> > If you are interested in getting notified of build failures and of further
> > changes in the packages you added or modified, please add yourself to the
> > DEVELOPERS file. This should be done in a separate patch of the series.  
> 
> Did I misunderstand this section, or should the documentation be updated?

The documentation should be updated. Back when we initially introduced
the DEVELOPERS file, we indeed thought that having a separate patch for
the DEVELOPERS file change was best, but we then changed our mind. I
thought we had updated our documentation, but apparently not. Do you
think you could send a patch fixing this?

> > It really doesn't have *any* runtime dependency on other Python  
> modules? I.e if you build just python-raven and the minimal python
> interpreter, it works?
> 
> Now that you mention it, it doesn't have any external requirements,
> but I'm unsure as to whether or not it requires any optional core
> modules. I would assume that it needs urllib at least. I'll test it
> and find out.

Enable just BR2_PACKAGE_PYTHON=y and BR2_PACKAGE_PYTHON_RAVEN=y, build
this, boot, and check that python-raven works correctly on the target.
That's the only way to be sure :)

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

      reply	other threads:[~2017-11-03 20:00 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-02 16:34 [Buildroot] [PATCH 1/2] package: add python-raven Joseph Kogut
2017-11-02 16:34 ` [Buildroot] [PATCH 2/2] DEVELOPERS: add myself as maintainer for python-raven Joseph Kogut
2017-11-02 20:32 ` [Buildroot] [PATCH 1/2] package: add python-raven Thomas Petazzoni
2017-11-03 19:08   ` Joseph Kogut
2017-11-03 20:00     ` Thomas Petazzoni [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=20171103210029.6ddf4546@windsurf \
    --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 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.