Hey Charlie,<br><br><div class="gmail_quote">2009/3/17 charliej <span dir="ltr"><<a href="mailto:cjsmo@cableone.net">cjsmo@cableone.net</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Mon, 2009-03-16 at 15:50 +0100, Arnaud Quette wrote:<br>
><br>
> 2009/3/14 charliej<br>
> Arnaud,<br>
><br>
> Hey Charlie,<br>
><br>
><br>
><br>
> Thank you for looking at this, IMHO this update will benefit<br>
> Ampache and<br>
> Rythmbox users. If there is any other way I can help out feel<br>
> free to<br>
> contact me.<br>
><br>
> sure, become a Debian Maintainer (or a DD) and adopt coherence,<br>
> please ;-)<br>
<br>
</div>Working on it! Actually I am thinking of going through the NM process.</blockquote><div><br>eh, nice ^_^<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;">
Having a problem of finding a Debian Member that lives close to me to<br>
get my key signed, but anyway that's a different can of worms.<br>
</blockquote><div><br>well, it's an important point since it officially confirms your digital identity for Debian!<br>you really have to put a high prio on this point and attend to an event or manage somehow to get a DD signing your key.<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;">
I would be willing to take over python-coherence, as long as I could<br>
call on you from time to time with questions/sponsorship of package<br>
updates. </blockquote><div><br>sure, I'm very opened to "training the new generation". it's part of a DD role ;-)<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;">
I am the current maintainer of Ampache and Ampache-themes.<br>
<div class="im">><br>
><br>
> more seriously, have you tested your patch?<br>
<br>
</div>yes, but I now understand why the install and test worked but the actual<br>
patch did not work.<br>
<div class="im"></div></blockquote><div><br>;-) you should never let false fixed go out. prefer to call to somebody or RFH if needed.<br>using pdebuild is helpful to get the latest lintian.<br>also see my below cdbs remark.<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;"><div class="im">
> I've noticed several things there:<br>
> - "DEB_INSTALL_ARGS += -XMochiKit" should be "DEB_DH_INSTALL_ARGS +=<br>
> -XMochiKit"<br>
<br>
</div>I am still unraveling the mysteries of CDBS and python packages. But<br>
after working on this package it makes more sense, but still have a lot<br>
to learn.<br>
<div class="im"></div></blockquote><div><br>an hint with cdbs is to directly look at .mk files. these are fairly well commented, and I found this more useful than the doc. the online examples can be helpful too.<br>for debhelpers (ie dh_install), it's /usr/share/cdbs/1/rules/<a href="http://debhelper.mk">debhelper.mk</a><br>
<br>be sure that we always have to learn. life wouldn't be fun otherwise ;-)<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;">
<div class="im">
> but will not be helpfull since the setup.py install the files directly<br>
> to python-coherence.<br>
> for the above to work, we should first go through debian/tmp using<br>
> DEB_DESTDIR</div></blockquote><div><br>I've missed to mention the quick solution:<br>binary-predeb/python-coherence::<br> /bin/rm -f debian/python-coherence/usr/share/pyshared/coherence/web/static/MochiKit.js<br>
<br>note that both solutions are fine here since the fix is limited to 1 file.<br>so the move to tmp not really needed.<br>if in the future there is a need to split and spawn new binary pkg, then you should go that way and use the .install files...<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;"><div class="im">
> - 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>
<br>
</div>I have pushed these changes to the launchpad branch.<br>
<div class="im"></div></blockquote><div><br>I've also noticed a strange .TH format (soft release instead of last edit date), but have not read it thoroughly .<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;">
<div class="im">
> - there is still a manpage missing for applet-coherence<br>
<br>
</div>my bad missed that one hmmm. Going to get started on this in the am,<br>
along with rebuilding/testing the package with the above changes. I<br>
will post back when the manpage and tests are complete.</blockquote><div><br>great, thanks.<br>note that as soon as your diff is validated, I'll upload the new release with an added Uploaders field for you ;-)<br>that will be the first step of the take over.<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;">
<div class="im">> I'm still working on this, but in very low prio background mode... but<br>
> would be interested in some feedback from you since I don't use<br>
> coherence nor the associated software...<br>
<br>
</div>That's fine, an associate program Ampache-3.5 is due out in about a<br>
month or so. Python-coherence makes use of Ampache's XML-API, and there<br>
will be some significant changes to the API with the 3.5 release, so<br>
this update of python-coherence is needed so coherence and ampache play<br>
nice. :) My intent, is to have both updated packages hit the archives<br>
at around the same time, if possible.</blockquote><div><br>leave a bit of time for the build-dep (ie coherence) to get buildd job's done, before sending ampache.<br>delaying by a day, and watching buildd logs to ensure everything is fine for coherence is a good idea.<br>
this will save build resources from failing to build to due the missing coherence dep ;-)<br><br></div></div>cheers,<br>Arnaud<br>