On Tuesday 02 March 2010 10:08:24, Raphael Hertzog wrote:
> Coming back to the topic, the dependency between desktopcouch and -records
> got added following https://bugs.launchpad.net/ubuntu/+bug/422179
> but Ken Vandine never gave the error message he had.
> 
> IMO it's this dependency that should not exist and I'm not sure why
> the service requires it. Maybe the part of -records that is required
> by desktopcouch should be moved to python-desktopcouch.

After Bill poked me about the bug not really being fixed by the package merge, 
I investigated a bit more about that LP bug, and the overall dependency 
situation.

The problem is that start_local_couchdb.py uses:

  from desktopcouch.records.server import CouchDatabase

which is available in the current python-desktopcouch-records package (that's 
why Ken added the dependency).

The server.py module needs server_base.py and record.py. This would leave 
/usr/share/pyshared/desktopcouch/records/ (in the py-desktopcouch-records 
package, at the moment) almost empty, with a major part of the package 
containing things that I wouldn't call "Records API" (thus the package name 
would be inappropriate, at least).

From:

$ dpkg -L python-desktopcouch-records  | grep pyshared/desktopcouch/
/usr/share/pyshared/desktopcouch/notes/[..]
/usr/share/pyshared/desktopcouch/contacts/[..]
/usr/share/pyshared/desktopcouch/replication_services/[..]
/usr/share/pyshared/desktopcouch/records/
/usr/share/pyshared/desktopcouch/records/server_base.py
/usr/share/pyshared/desktopcouch/records/record.py
/usr/share/pyshared/desktopcouch/records/couchgrid.py
/usr/share/pyshared/desktopcouch/records/__init__.py
/usr/share/pyshared/desktopcouch/records/tests
/usr/share/pyshared/desktopcouch/records/tests/[..]
/usr/share/pyshared/desktopcouch/records/field_registry.py
/usr/share/pyshared/desktopcouch/records/server.py
$

It would become something like (stripping the files that would need to go to 
the "desktopcouch" package):

/usr/share/pyshared/desktopcouch/notes/[..]
/usr/share/pyshared/desktopcouch/contacts/[..]
/usr/share/pyshared/desktopcouch/replication_services/[..]
/usr/share/pyshared/desktopcouch/records/
/usr/share/pyshared/desktopcouch/records/couchgrid.py
/usr/share/pyshared/desktopcouch/records/tests
/usr/share/pyshared/desktopcouch/records/tests/[..]
/usr/share/pyshared/desktopcouch/records/field_registry.py


I think a "proper" solution could be:
 1) put /usr/share/pyshared/desktopcouch/records/ into "desktopcouch" too (and
    add Provides: python-desktopcouch-records)
 2) move the remaining /usr/share/pyshared/desktopcouch/*/ to a new "python- 
    desktopcouch-something" package (where something == "extra", or similar).

What do you think?


Kindly,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 ----|----
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to