Bug#287157: #287157 - An example to this bug

2006-02-13 Thread Nelson A. de Oliveira
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi!

Peter Eisentraut wrote:
 Nelson A. de Oliveira wrote:
 
The makefile sets a CFLAGS containing this:

CFLAGS = -I$(ZDIR) -I$(PNGDIR) -Wall $(CRELEASE)

Since CDBS passes CFLAGS=-g -Wall -O2, it overrides the CFLAGS
included on the source code, making the package fail to build.
 
 
 Makefiles written in this way are broken -- at least with respect to 
 common build system practices.  There are packages on the other hand 
 that do it correctly, namely set CFLAGS in the makefiles expecting them 
 to be overridden if desired on the command line, and if we made the 
 change that is proposed in this bug we would break those packages.

Good. I have contacted the upstream author of this program and sent him
a patch to fix this issue.

About this bug, maybe tags it wont-fix, so people will see that actually
it's not a bug on CDBS? (if we left it open, people can think that
nothing is being done about this and if we close the bug, someone can
open another similar bug later). Just a suggestion.

Thank you!

Best regards,
Nelson

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.2 (GNU/Linux)

iD8DBQFD8UMfAQwuptkwlkQRAsQjAJ9h0jqrF0GD7ACUI5RIO/RR6yPi1gCfT1nc
28fcL6dC/fPBUvV9msLTSbo=
=d5dp
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#287157: #287157 - An example to this bug

2006-02-10 Thread Peter Eisentraut
Nelson A. de Oliveira wrote:
 The makefile sets a CFLAGS containing this:

 CFLAGS = -I$(ZDIR) -I$(PNGDIR) -Wall $(CRELEASE)

 Since CDBS passes CFLAGS=-g -Wall -O2, it overrides the CFLAGS
 included on the source code, making the package fail to build.

Makefiles written in this way are broken -- at least with respect to 
common build system practices.  There are packages on the other hand 
that do it correctly, namely set CFLAGS in the makefiles expecting them 
to be overridden if desired on the command line, and if we made the 
change that is proposed in this bug we would break those packages.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#287157: #287157 - An example to this bug

2006-01-25 Thread Nelson A. de Oliveira
Hi!

I reached this bug yesterday.
If you could take a look at #349699, I have explained what happened to me.

The makefile sets a CFLAGS containing this:

CFLAGS = -I$(ZDIR) -I$(PNGDIR) -Wall $(CRELEASE)

Since CDBS passes CFLAGS=-g -Wall -O2, it overrides the CFLAGS
included on the source code, making the package fail to build.

Nelson