Bug 405133 – media-gfx/mcomix should depend on dev-lang/python[sqlite]

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description RazrFalcon 2012-02-21 11:46:17 UTC]

Created attachment 302685 [details]
emerge --info

mcomix didn't start without python sqlite module.

$ mcomix
Traceback (most recent call last):
  File "/usr/bin/mcomix-2.7", line 9, in <module>
    load_entry_point('mcomix==0.97.1', 'console_scripts', 'mcomix')()
  File "/usr/lib64/python2.7/site-packages/pkg_resources.py", line 338, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib64/python2.7/site-packages/pkg_resources.py", line 2280, in load_entry_point
    return ep.load()
  File "/usr/lib64/python2.7/site-packages/pkg_resources.py", line 1990, in load
    entry = __import__(self.module_name, globals(),globals(), ['__name__'])
  File "/usr/lib64/python2.7/site-packages/mcomix/run.py", line 158, in <module>
    from mcomix import main
  File "/usr/lib64/python2.7/site-packages/mcomix/main.py", line 15, in <module>
    from mcomix import file_handler
  File "/usr/lib64/python2.7/site-packages/mcomix/file_handler.py", line 21, in <module>
    from mcomix import last_read_page
  File "/usr/lib64/python2.7/site-packages/mcomix/last_read_page.py", line 5, in <module>
    import sqlite3
ImportError: No module named sqlite3

[Comment 1 Ryan Hill (RETIRED) 2012-02-22 00:53:06 UTC]

It does depend on dev-lang/python[sqlite].

 RDEPEND="${DEPEND}
     app-arch/unrar
     >=dev-python/imaging-1.1.5
     >=dev-python/pygtk-2.14
     virtual/jpeg
     || ( dev-lang/python[sqlite] dev-python/pysqlite )
     !media-gfx/comix"

Maybe you have a different version of python installed that satisfies the dependency?

[Comment 2 Samuli Suominen (RETIRED) 2012-02-22 15:00:30 UTC]

(In reply to comment #1)
> It does depend on dev-lang/python[sqlite].

It should use python.eclass and

PYTHON_USE_WITH="sqlite"

Look for eg. games-kids/gcompris ebuild

[Comment 3 Ryan Hill (RETIRED) 2012-02-23 05:57:32 UTC]

Right, but the any-of dependency throws a wrench into things.  We'd have to add a pysqlite USE flag and use PYTHON_USE_WITH_OPT.  I'm not sure I like that idea.  There's also another bug here in that last_read_page doesn't fall back to pysqlite if the python sqlite module is missing.  I'm inclined to just drop the pysqlite option altogether.

[Comment 4 Samuli Suominen (RETIRED) 2012-02-23 06:15:56 UTC]

(In reply to comment #3)
> Right, but the any-of dependency throws a wrench into things.  We'd have to add
> a pysqlite USE flag and use PYTHON_USE_WITH_OPT.  I'm not sure I like that
> idea.  There's also another bug here in that last_read_page doesn't fall back
> to pysqlite if the python sqlite module is missing.  I'm inclined to just drop
> the pysqlite option altogether.

That's what I've been doing; forcing the USE="sqlite" enabled for python instead of supporting the separate pysqlite. I've been in same situation with multiple packages. Not perfect, but imho best we can do :/

[Comment 5 Ryan Hill (RETIRED) 2012-02-24 04:47:16 UTC]

Fixed.