[Pkg-utopia-maintainers] Bug#518391: Patches for coherence

[charliej]

On Mon, 2009-03-16 at 15:50 +0100, Arnaud Quette wrote:
> 
> 2009/3/14 charliej 
>         Arnaud,
> 
> Hey Charlie,
>  
> 
>         
>         Thank you for looking at this, IMHO this update will benefit
>         Ampache and
>         Rythmbox users.  If there is any other way I can help out feel
>         free to
>         contact me.
> 
> sure, become a Debian Maintainer (or a DD) and adopt coherence,
> please ;-)

Working on it!  Actually I am thinking of going through the NM process.
Having a problem of finding a Debian Member that lives close to me to
get my key signed, but anyway that's a different can of worms.

I would be willing to take over python-coherence, as long as I could
call on you from time to time with questions/sponsorship of package
updates.  I am the current maintainer of Ampache and Ampache-themes.
> 
> 
> more seriously, have you tested your patch?

yes, but I now understand why the install and test worked but the actual
patch did not work. 

> I've noticed several things there:
> - "DEB_INSTALL_ARGS += -XMochiKit" should be "DEB_DH_INSTALL_ARGS +=
> -XMochiKit"

I am still unraveling the mysteries of CDBS and python packages.  But
after working on this package it makes more sense, but still have a lot
to learn.

> but will not be helpfull since the setup.py install the files directly
> to python-coherence.
> for the above to work, we should first go through debian/tmp using
> DEB_DESTDIR
> - the added Build-dep doesn't add the coma to the previous entry,
> - there was a typo in the added manpage (AVAILAB*L*E STORES)

I have pushed these changes to the launchpad branch.

> - there is still a manpage missing for applet-coherence

my bad missed that one hmmm.  Going to get started on this in the am,
along with rebuilding/testing the package with the above changes.  I
will post back when the manpage and tests are complete.
> 
> I'm still working on this, but in very low prio background mode... but
> would be interested in some feedback from you since I don't use
> coherence nor the associated software...

That's fine, an associate program Ampache-3.5 is due out in about a
month or so.  Python-coherence makes use of Ampache's XML-API, and there
will be some significant changes to the API with the 3.5 release, so
this update of python-coherence is needed so coherence and ampache play
nice. :)  My intent, is to have both updated packages hit the archives
at around the same time, if possible.
> 

Cheers
Charlie
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : http://lists.alioth.debian.org/pipermail/pkg-utopia-maintainers/attachments/20090316/252ac68e/attachment.pgp