Bug 452126

Summary:	app-misc/gnomecatalog: uses dev-python/pysqlite:2 instead of built-in sqlite3
Product:	Gentoo Linux	Reporter:	Michał Górny <mgorny>
Component:	Current packages	Assignee:	No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status:	RESOLVED FIXED
Severity:	QA	CC:	python
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	452122

Description Michał Górny archtester

2013-01-14 21:54:26 UTC

The package does depend on dev-python/pysqlite:2 but not on Python implementation USE=sqlite.

The pysqlite package is no longer maintained and compatible with Python 2 only; while the builtin 'sqlite3' module is basically a maintained clone/fork of pysqlite.

If the package actually imports pysqlite2 unconditionally, please check whether it wouldn't work with the builtin 'sqlite3' module as well. If it does, upstream should probably fix the package to import 'sqlite3' *at least* when 'pysqlite2' is not available. If it doesn't, please let us know and we will know that pysqlite needs to be kept in the tree.

If the package simply supports 'sqlite3', please let us know or update the deps. However, the latter is unnecessary since the 'old' Python eclass doesn't provide a proper way to do this, and we may decide to drop pysqlite entirely after getting feedback for the packages depending on it.

Comment 1 Michał Górny archtester

2013-01-15 22:58:41 UTC

Grepping the code shows that the package imports 'pysqlite2' only. Need to investigate further.

Comment 2 Michał Górny archtester

2013-01-16 20:54:35 UTC

(In reply to comment #1)
> Grepping the code shows that the package imports 'pysqlite2' only. Need to
> investigate further.

Well, I seem to be able to run it successfully with 'sqlite3' replacing 'pysqlite2'. Well, as successfully as with 'pysqlite2', at least...

Comment 3 Michał Górny archtester

2013-01-17 15:57:40 UTC

Fix committed. Not upstreamed since seems that package is not maintained anymore. I'd vote for lastriting it.