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

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



On Sun, Jul 01, 2012 at 04:37:44PM +0100, Adam D. Barratt wrote:
> On Sun, 2012-07-01 at 16:48 +0300, Tzafrir Cohen wrote:
> > Please unblock package spandsp
> > 
> > See http://bugs.debian.org/679736 . The test suite may fail when building
> > in parallel. This patch avoids building the test suite in parallel.

Updated the issue and uploaded 0.0.6~pre20-3 .

> 
> So far as I can tell, that bug does not affect wheezy, as it was
> introduced in a version of the package which has only ever been in
> unstable; is that correct?

The bug was introduced by the fact that as of 0.0.6pre20 I enabled the
test suite at build time. I suppose that if I enabled the test suite on
pre18 (in Wheezy now) it would have the same effect. Unfortunately it
took me a while to notice the issue after uploading the package to
Unstable.

> 
> On that basis, the changes from both -1 and -2 are relevant to the
> unblock.  While the changes between the current testing package and -1
> aren't huge, the changelog isn't really very helpful in terms of helping
> me decide whether they're appropriate for an unblock as it just says
> "new upstream version".
> 
> Would it be possible to expand (briefly should be fine) on what the
> changes in the new upstream version are?

As of now there are two separate issues:

1. The test suite was not designed to be built in parallel:
   https://buildd.debian.org/status/logs.php?pkg=spandsp&ver=0.0.6~pre20-1
   All failiures are due to a test file not having a proper input, as
   it was deleted or being regenerated by another program running the same
   thing.

Fix for that: .NOTPARALLEL in tests/Makefile . In -2 it was not done
properly and hence didn't really work. Works now.

2. Something that at first glance as a mips toolchain issue:
   https://buildd.debian.org/status/logs.php?pkg=spandsp&arch=mips
   Note that the builds for 0.0.6~pre20-2 and 0.0.6~pre20-3 failed much
   earlier than the one for 0.0.6~pre20-1 (which was also built on a
   different buildd).

Looking into that one.

-- 
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: