Re: Request for voluntary package reviews
- To: chrysn <firstname.lastname@example.org>, email@example.com
- Subject: Re: Request for voluntary package reviews
- From: Alexandre Ratchov <firstname.lastname@example.org>
- Date: Mon, 8 Nov 2010 18:45:18 +0100
- Message-id: <20101108174518.GX5295@moule.localdomain>
- In-reply-to: <20101107202351.GE38432@l04.local>
- References: <20101030194934.GD6538@l04.lan> <20101030121441.GA15244@moule.localdomain> <20101030220327.GA27592@hephaistos.amsuess.com> <20101031120631.GC15244@moule.localdomain> <20101107202351.GE38432@l04.local>
On Sun, Nov 07, 2010 at 09:23:53PM +0100, Michael Tautschnig wrote:
> Hi Alexandre,
> [...] (review by chrysn and comments)
> > >
> > > i couldn't test the functionality itself for lack of midi hardware, but
> > > at least that's reflected by appropriate warnings by midish.
> > I was mostly concerned by any packaging mistakes, but testing the code
> > itself never hurts :-)
> The package includes regression tests - would it make sense to run these at
> build time, or would that only make sense if a MIDI device was present? I tried
> to run them and all of them passed, but I don't really know if this is just
> trivially true as I don't have MIDI hardware. If it makes sense to run them even
> without MIDI hardware, please include a make check run in future releases of
> your package.
Indeed these tests require no midi hardware. They are mostly useful to
developpers but running them can't hurt ;) I'll include them in the
next version of the package.
> > Thanks for your comments, the new package is on the usual place:
> > http://mentors.debian.net/debian/pool/main/m/midish
> Thanks to the review by chrysn it was pretty easy to do another final review; I
> have now built and uploaded your package. For future versions, please take the
> following into consideration:
> - As said above, a regression test would of course be nice to have.
> - In line with  you might consider acknowleding reviews such as chrysn's as
> those help in speeding up the sponsoring process.
> - You require a compat level of 7, so you could also make use of the much
> simplified rules file made possible by debhelper 7. See  for further
Thanks for the review and for uploading the package. I just got a
notification that the package fails to build on kfreebsd, I guess the
kfreebsd has clock_gettime() in librt. Fixing these issues...