[SCM] mplayer packaging branch, ubuntu, updated. debian/1.0.rc3++final-0ubuntu2

[Reinhard Tartler]

On Do, Jun 03, 2010 at 17:08:53 (CEST), Felipe Sateler wrote:

> On Thu, Jun 3, 2010 at 09:06,  <siretart at users.alioth.debian.org> wrote:
>> +# Support multiple makes at once
>> +ifneq (,$(filter parallel=%,$(DEB_BUILD_OPTIONS)))
>> +NUMJOBS = -j$(patsubst parallel=%,%,$(filter parallel=%,$(DEB_BUILD_OPTIONS)))
>> +else
>> +# on i386 and amd64, we query the system unless overriden by DEB_BUILD_OPTIONS
>> +ifeq      ($(DEB_HOST_ARCH),i386)
>> +NUMJOBS := -j$(shell getconf _NPROCESSORS_ONLN 2>/dev/null || echo 1)
>> +else ifeq ($(DEB_HOST_ARCH),amd64)
>> +NUMJOBS := -j$(shell getconf _NPROCESSORS_ONLN 2>/dev/null || echo 1)
>> +endif
>> +endif
>> +
>
> For this particular case, I think what you actually want is
> DEB_BUILD_ARCH.

correct. I'll fix this in both the mplayer and ffmpeg package.

> However, I'm not sure enabling parallel building by default is a good
> idea.

There have been no complaints this far and makes building packages on my
machines faster. 

> For example, buildds probably do more than one build at a time.

Really? AFAIUI, at least buildd doesn't support this OOTB.

I'd rather leave that in until a buildd admin asks us to stop doing that.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4