[ubuntu-dev] Bug#765151: gdebi: FTBFS: Tests failures

[Ivo De Decker]

Control: tags -1 patch

Hi,

On Mon, Oct 13, 2014 at 07:09:22PM +0200, David Suárez wrote:
> During a rebuild of all packages in sid, your package failed to build on
> amd64.
> 
> Relevant part (hopefully):
> >   W: error-package: no-section-field
> >   W: error-package: no-priority-field
> >   E: error-package: wrong-file-owner-uid-or-gid etc/ 1000/1000
> >   E: error-package: wrong-file-owner-uid-or-gid etc/foo 1000/1000
> >   W: error-package: maintainer-script-ignores-errors postinst
> >   
> > - Lintian finished with exit status 0?                                   ^
> > + Lintian finished with exit status 1?                                   ^

This bug seems to be caused by an error in the way gdebi handles the lintian
output. The code adds watchers for both stdin and stderr, and adds watchers to
get the exit code in the same loop. As this is done twice, there is a race
condition between both watchers. It seems one of them doesn't get the exit
code and gets 0 instead. If that one runs first, the test fails. By adding the
watcher outside the loop that adds the IO watchers, there is only a single
watcher, which fixes the build.

The attached patch does just that. If there is no objection, I'm happy to do
an NMU with this patch.

Cheers,

Ivo


-------------- next part --------------
A non-text attachment was scrubbed...
Name: gdebi_765151.patch
Type: text/x-diff
Size: 1649 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/ubuntu-dev-team/attachments/20141109/3afae058/attachment.patch>