On Fri, Oct 29, 2010 at 10:42 PM, Nanakos Chrysostomos <[email protected]> wrote: > On Fri, Oct 29, 2010 at 08:02:33PM +0700, Theppitak Karoonboonyanan wrote: >> On Fri, Oct 29, 2010 at 7:46 PM, Theppitak Karoonboonyanan >> <[email protected]> wrote: >> > - The package doesn't build without pulseaudio installed: >> > ---8<--- >> > ln qso.d/QSO ./QSO >> > (cat test_input; qso.d/QSO) | ./morse -w 24 -l -e >> > Could not initialize audio: Connection refused >> > Can't access speaker. >> > make[2]: *** [testmorse] Error 1 >> > ---8<--- >> > Just libpulse-dev b-dep seems insufficient, and as the program won't work >> > without pulseaudio, this means it should also be a dependency of the binary >> > package. > Fixed. >> > >> > Alternatively, is it possible to make pulseaudio optional rather than >> > mandatory? > Fixed. > >> One more thing, the "install" target is missing in upstream Makefile. >> So, the "install" stage in debian/rules also fails. > Fixed. > > Re-uploaded the fixed package to mentors.d.n.
Your fixes look good. But repacking pristine tarball is not a good practice. Please use debian/patches/ instead, as your package is already in "3.0 (quilt)" format. In doing so, please just use the downloaded upstream tarball as-is (no need to rename the directory to morse-2.2-orig), then *rename* (not repack) the tarball to morse_2.2.orig.tar.gz. Please make sure not to modify source files directly. Always use quilt to produce patches. In previous version, 2.1-4, you had 6 patches in debian/patches/. Is there a good reason to drop all of them? If so, please log it in your debian/changelog. Or you may resurrect some patches, such as 00makefile, 01morseX11 you are currently applying. In fact, the patch to add morseX11.1 and morseLinux.1 manpages can be replaced by adding them under debian/ and use dh_installman (and the list in debian/manpages) to install them. The patches from previous version, however, refer to the ITA bug (http://bugs.debian.org/553991) which is not relevant to the problem they are fixing. So, if you are using them in the new version, please just drop the Bug-Debian: field if it does not actully fix the bug. After all, don't forget to log changes from previous version you have done, including the added Recommends: and debian/rules changes. Regards, -- Theppitak Karoonboonyanan http://linux.thai.net/~thep/ -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected] Archive: http://lists.debian.org/[email protected]
