[PATCH] perfcounter: don't keep warnings treated as errors as default

From: Frederic Weisbecker
Date: Thu Jun 04 2009 - 13:15:03 EST


Building userspace perfcounter tools gives such warning:

cc1: warnings being treated as errors
builtin-top.c: In function âprint_sym_tableâ:
builtin-top.c:207: erreur: ignoring return value of âwriteâ, declared with attribute warn_unused_result
make: *** [builtin-top.o] Erreur 1

It's about checking if a write was successful on stdout.
The real point here is that treating each warning as errors by default
can be avoided, especially for such harmless warning.

This patch drop -Werror from perfcounter tools makefile.

Signed-off-by: Frederic Weisbecker <fweisbec@xxxxxxxxx>
---
Documentation/perf_counter/Makefile | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/Documentation/perf_counter/Makefile b/Documentation/perf_counter/Makefile
index c9ec458..9b42897 100644
--- a/Documentation/perf_counter/Makefile
+++ b/Documentation/perf_counter/Makefile
@@ -159,7 +159,7 @@ uname_V := $(shell sh -c 'uname -v 2>/dev/null || echo not')

# CFLAGS and LDFLAGS are for the users to override from the command line.

-CFLAGS = -ggdb3 -Wall -Werror -Wstrict-prototypes -Wmissing-declarations -Wmissing-prototypes -std=gnu99 -Wdeclaration-after-statement -O6 -march=native
+CFLAGS = -ggdb3 -Wall -Wstrict-prototypes -Wmissing-declarations -Wmissing-prototypes -std=gnu99 -Wdeclaration-after-statement -O6 -march=native
LDFLAGS = -lpthread -lrt -lelf
ALL_CFLAGS = $(CFLAGS)
ALL_LDFLAGS = $(LDFLAGS)
--
1.6.2.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/