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

[Jonas Smedegaard]

On Thu, Jun 03, 2010 at 01:06:31PM +0000, 
siretart at users.alioth.debian.org wrote:
>The following commit has been merged in the ubuntu branch:

>+# 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

Not sure, but it looks to me that above not only provides _support_ for 
parallel builds but also _enable_ it by default.

I don't know if Ubuntu (which apparently you are targeting here) has 
different poicy, but I believe above is illegal for Debian: According to 
Debian Policy 3.8.4 §4.9.1, we should enable parallel builds if possible 
but leave it "to the package maintainer to decide whether the package 
build times are long enough and the package build system is robust 
enough to make supporting parallel builds worthwhile."


Kind regards,

  - Jonas

-- 
  * Jonas Smedegaard - idealist & Internet-arkitekt
  * Tlf.: +45 40843136  Website: http://dr.jones.dk/

  [x] quote me freely  [ ] ask before reusing  [ ] keep private
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20100604/313fac58/attachment.pgp>