[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: morse (New upstream release)

On Fri, Oct 29, 2010 at 10:42 PM, Nanakos Chrysostomos
<nanakos@wired-net.gr> 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
>> <thep@debian.org> 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.

Theppitak Karoonboonyanan

Reply to: