Bug#758531: Better looking patch

Krzysztof A. Sobiecki sobkas at gmail.com
Wed Aug 20 19:12:30 UTC 2014


Felipe Sateler <fsateler at debian.org> writes:

> (Why did you not CC the bug? Please CC the bug if not intentional)
>
> On Wed, Aug 20, 2014 at 10:46 AM, Krzysztof A. Sobiecki
> <sobkas at gmail.com> wrote:
>> Felipe Sateler <fsateler at debian.org> writes:
>>
>>> Hi,
>>>
>>> On Wed, Aug 20, 2014 at 8:58 AM, Krzysztof A. Sobiecki <sobkas at gmail.com> wrote:
>>>> A better looking patch to disable parallel builds
>>>>
>>>> --- pulseaudio-5.0.orig/src/Makefile.am
>>>> +++ pulseaudio-5.0/src/Makefile.am
>>>> @@ -2195,3 +2195,4 @@ coverage:
>>>>  endif
>>>>
>>>>  .PHONY: massif update-all update-ffmpeg update-map-file coverage
>>>> +.NOTPARALLEL:
>>>
>>> Why disable parallel building?
>>>
>> Because it causes a build failure? I think?
>
> I assume that is the proximate cause. But building in parallel works
> here and in porterboxes, so I thought you could shed some light on
> what is exactly failing.
 To be precise it's install process that fails while parallel make is
used. I think that this causes my problem:
install-modlibexecLTLIBRARIES is run with other targets in install-exec-am
and not all libraries were copied in time to debian/tmp before install-modlibexecLTLIBRARIES makes
libtool relink some other libraries.

 The fact that linker sometimes complained only about one library make
it almost clear that it's a race condition. Build process worked because
install-libLTLIBRARIES was faster than install-modlibexecLTLIBRARIES.


To make it less luck based, install-libLTLIBRARIES must be run before
install-modlibexecLTLIBRARIES.

That is why parallel build might fail sometimes.
Also I know nothing about autotools.

>
> In the mean time, please stick with the unstable package. THe one in
> experimental is work in progress.
To miss all the fun?

-- 
X was an interactive protocol: 
alpha blending a full-screen image looked like slugs racing down the monitor. 
http://www.keithp.com/~keithp/talks/usenix2000/render.html



More information about the pkg-pulseaudio-devel mailing list