[SCM] x264/master: Force CFLAGS to be set by user via DEB_CFLAGS instead of by dpkg-buildpackage.

[Jonas Smedegaard]

On Wed, Oct 27, 2010 at 08:03:37PM +0000, ceros-guest at users.alioth.debian.org wrote:
>    Force CFLAGS to be set by user via DEB_CFLAGS instead of by dpkg-buildpackage.


>+# Set CFLAGS from DEB_CFLAGS if defined, otherwise simply add -Wall
>+CFLAGS = $(shell test -z "$$DEB_CFLAGS" || echo "$$DEB_CFLAGS")
>+ifeq (,$(CFLAGS))
>+  CFLAGS = -Wall
>+endif

If I understand your intention correctly, this is more elegant:

CFLAGS = $(or $(DEB_CFLAGS),-Wall)


  - 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/20101027/0e0aec9d/attachment.pgp>