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

Bug#679778: unblock: spandsp/0.0.6~pre20-2



Hi

(I was sure this was sent before. Sorry for the delay)

On Tue, Jul 03, 2012 at 08:31:48PM +0100, Adam D. Barratt wrote:
> On Mon, 2012-07-02 at 22:39 +0000, Tzafrir Cohen wrote:
> > There's no real changelog. However the difference between pre18 and
> > pre20 is not big.
> >
> > pre19: https://gitorious.org/spandsp/spandsp/commit/926c6eec0a2955dffccea3992232d0a064f4a5f6
> > A bugfix commit. IIRC the fix in t4_rx.c was discovered by faxes that
> > managed to crash Asterisk.
> > 
> > 
> > pre20: https://gitorious.org/spandsp/spandsp/commit/83f249c1a0172d3246a2ffb69fb534f725bdbca3
> > Again: bugfixes. Sadly I'm not sure what for, exactly
> 
> Hmmm.  I'm prepared to go with the change in pre19 being suitable, and
> give the benefit of the doubt to the pre20 changes and hope they don't
> turn out to be a pain.  There are a couple of packaging changes that
> don't appear to be mentioned in the changelog though:
> 
> debian/control:
> -Conflicts: asterisk-app-dtmftotext (<= 0.0.20060218-4)

That package is long gone (since Etch). It also depended on libspandsp1
which is gone (assuming you don't use m68k). So there's no longer a need
for the explicit Conflicts.

> 
> debian/rules:
>  %:
> -       dh $@ --with-autotools-dev --parallel
> +       dh $@ --parallel

I guess it should have been '--with autotools-dev'. But as-is it seems a
no-op. So I preffered to just remove it rather than change packaging
further.

-- 
Tzafrir Cohen         | tzafrir@jabber.org | VIM is
http://tzafrir.org.il |                    | a Mutt's
tzafrir@cohens.org.il |                    |  best
tzafrir@debian.org    |                    | friend


Reply to: