Re: [SCM] xvidcore/master: Set CFLAGS via DEB_CFLAGS, otherwise let xvidcore's build system chose CFLAGS.

2010-10-29 Thread Fabian Greffrath

Am 28.10.2010 09:13, schrieb Fabian Greffrath:

By the way, what's the purpose of these changes at all?!


Again, which tool sets DEB_CFLAGS, what are they used for and why 
aren't they just added via CFLAGS += DEB_CFLAGS (or maybe "?=")?


 - Fabian

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: [SCM] xvidcore/master: Set CFLAGS via DEB_CFLAGS, otherwise let xvidcore's build system chose CFLAGS.

2010-10-28 Thread Fabian Greffrath

Am 28.10.2010 07:32, schrieb Jonas Smedegaard:

+# Set CFLAGS from DEB_CFLAGS if defined, otherwise let xvidcore's
build system
+# set CFLAGS to use.
+CFLAGS = $(or $(DEB_CFLAGS),)

Simpler:
CFLAGS = $(DEB_CFLAGS)


By the way, what's the purpose of these changes at all?!

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: [SCM] xvidcore/master: Set CFLAGS via DEB_CFLAGS, otherwise let xvidcore's build system chose CFLAGS.

2010-10-27 Thread Jonas Smedegaard

On Wed, Oct 27, 2010 at 10:34:46PM +, ceros-gu...@users.alioth.debian.org 
wrote:

   Set CFLAGS via DEB_CFLAGS, otherwise let xvidcore's build system chose 
CFLAGS.

diff --git a/debian/rules b/debian/rules
index 137d4c9..c81bae1 100755
--- a/debian/rules
+++ b/debian/rules
@@ -1,4 +1,8 @@
#!/usr/bin/make -f

+# Set CFLAGS from DEB_CFLAGS if defined, otherwise let xvidcore's build system
+# set CFLAGS to use.
+CFLAGS = $(or $(DEB_CFLAGS),)
+


Simpler:

CFLAGS = $(DEB_CFLAGS)


 - Jonas

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

 [x] quote me freely  [ ] ask before reusing  [ ] keep private


signature.asc
Description: Digital signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers