All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: bitbake-devel@lists.openembedded.org
Subject: Re: [PATCH 2/2] prserv: fix import of sqlite3
Date: Fri, 20 Jan 2012 18:01:06 +0100	[thread overview]
Message-ID: <20120120170106.GC3645@jama.jama.net> (raw)
In-Reply-To: <1327077887.4268.18.camel@ted>

[-- Attachment #1: Type: text/plain, Size: 1538 bytes --]

On Fri, Jan 20, 2012 at 04:44:47PM +0000, Richard Purdie wrote:
> On Sun, 2012-01-15 at 22:03 +0100, Martin Jansa wrote:
> > * this is used in all other bitbake parts where sqlite3 is used, don't
> >   know why it wasn't used here, but it fails e.g. on Gentoo
> >   Traceback (most recent call last):
> >     File "bin/bitbake", line 39, in <module>
> >       from bb import cooker
> >     File "lib/bb/cooker.py", line 39, in <module>
> >       import prserv.serv
> >     File "lib/prserv/serv.py", line 4, in <module>
> >       import xmlrpclib,sqlite3
> >   ImportError: No module named sqlite3
> > 
> > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> > ---
> >  lib/prserv/serv.py |    7 ++++++-
> >  1 files changed, 6 insertions(+), 1 deletions(-)
> 
> Merged to master along with 1/2.
> 
> Why this is needed I'm not sure since I thought sqlite3 was included in
> python 2.6 onwards. Since you're running into it and it doesn't hurt
> anything I've merged it though.

In gentoo there is USE flag for sqlite support which is off by default,
so it's easy to get even newer python like 2.7 or 3* without sqlite
enabled by default and extra package dev-python/pysqlite providing
alternative.

I guess correct fix would be to change bitbake .ebuild to depend on
python with sqlite USE flag enabled, but as it was catched and fixed in
other places in bitbake code I've added it here too.

Cheers,

> 
> Cheers,
> 
> Richard
> 

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

      reply	other threads:[~2012-01-20 17:08 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-15 21:03 [PATCH 1/2] setup.py: install prserv too Martin Jansa
2012-01-15 21:03 ` [PATCH 2/2] prserv: fix import of sqlite3 Martin Jansa
2012-01-20 16:44   ` Richard Purdie
2012-01-20 17:01     ` Martin Jansa [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=20120120170106.GC3645@jama.jama.net \
    --to=martin.jansa@gmail.com \
    --cc=bitbake-devel@lists.openembedded.org \
    --cc=richard.purdie@linuxfoundation.org \
    /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.