<br><div class="gmail_quote">2009/3/14 charliej <br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Arnaud,<br>
</blockquote><div><br>Hey Charlie,<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
Thank you for looking at this, IMHO this update will benefit Ampache and<br>
Rythmbox users. If there is any other way I can help out feel free to<br>
contact me.<br>
</blockquote><div><br>sure, become a Debian Maintainer (or a DD) and adopt coherence, please ;-)<br></div></div><br>more seriously, have you tested your patch?<br>I've noticed several things there:<br>- "DEB_INSTALL_ARGS += -XMochiKit" should be "DEB_DH_INSTALL_ARGS += -XMochiKit"<br>
but will not be helpfull since the setup.py install the files directly to python-coherence.<br>for the above to work, we should first go through debian/tmp using DEB_DESTDIR<br>- the added Build-dep doesn't add the coma to the previous entry,<br>
- there was a typo in the added manpage (AVAILAB*L*E STORES)<br>- there is still a manpage missing for applet-coherence<br clear="all"><br>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...<br>
<br>cheers,<br>Arnaud<br>-- <br>Linux / Unix Expert R&D - Eaton - <a href="http://www.eaton.com/mgeops" target="_blank">http://www.eaton.com/mgeops</a><br>
Network UPS Tools (NUT) Project Leader - <a href="http://www.networkupstools.org/" target="_blank">http://www.networkupstools.org/</a><br>Debian Developer - <a href="http://people.debian.org/%7Eaquette/" target="_blank">http://people.debian.org/~aquette/</a><br>
Free Software Developer - <a href="http://arnaud.quette.free.fr/" target="_blank">http://arnaud.quette.free.fr/</a><br><br>