WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-changelog

[Xen-changelog] The attached patch adds -Werror to HOSTCFLAGS in Config.

To: xen-changelog@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-changelog] The attached patch adds -Werror to HOSTCFLAGS in Config.mk, makes
From: Xen patchbot -unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>
Date: Mon, 08 Aug 2005 04:32:09 -0400
Delivery-date: Mon, 08 Aug 2005 08:32:43 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-changelog-request@lists.xensource.com?subject=help>
List-id: BK change log <xen-changelog.lists.xensource.com>
List-post: <mailto:xen-changelog@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=unsubscribe>
Reply-to: xen-devel@xxxxxxxxxxxxxxxxxxx
Sender: xen-changelog-bounces@xxxxxxxxxxxxxxxxxxx
# HG changeset patch
# User kaf24@xxxxxxxxxxxxxxxxxxxx
# Node ID 41ceeb6828b57b082735518431cc67bce7e8916d
# Parent  c4512592a1dc11c76b94d87f14849bcc38871f72
The attached patch adds -Werror to HOSTCFLAGS in Config.mk, makes
xen/tools actually use HOSTCFLAGS (it was already using HOSTCC), and
fixes some gcc-4.0 signedness warnings in xen/tools/symbols.c.

Signed-off-by: Josh Triplett <josht@xxxxxxxxxx>

diff -r c4512592a1dc -r 41ceeb6828b5 Config.mk
--- a/Config.mk Mon Aug  8 08:18:38 2005
+++ b/Config.mk Mon Aug  8 08:32:52 2005
@@ -7,7 +7,7 @@
 
 # Tools to run on system hosting the build
 HOSTCC     = gcc
-HOSTCFLAGS = -Wall -Wstrict-prototypes -O2 -fomit-frame-pointer 
+HOSTCFLAGS = -Wall -Werror -Wstrict-prototypes -O2 -fomit-frame-pointer
 
 AS         = $(CROSS_COMPILE)as
 LD         = $(CROSS_COMPILE)ld
diff -r c4512592a1dc -r 41ceeb6828b5 xen/tools/Makefile
--- a/xen/tools/Makefile        Mon Aug  8 08:18:38 2005
+++ b/xen/tools/Makefile        Mon Aug  8 08:32:52 2005
@@ -10,4 +10,4 @@
        rm -f *.o symbols
 
 symbols: symbols.c
-       $(HOSTCC) -o $@ $<
+       $(HOSTCC) $(HOSTCFLAGS) -o $@ $<
diff -r c4512592a1dc -r 41ceeb6828b5 xen/tools/symbols.c
--- a/xen/tools/symbols.c       Mon Aug  8 08:18:38 2005
+++ b/xen/tools/symbols.c       Mon Aug  8 08:32:52 2005
@@ -152,8 +152,8 @@
        /* include the type field in the symbol name, so that it gets
         * compressed together */
        s->len = strlen(str) + 1;
-       s->sym = (char *) malloc(s->len + 1);
-       strcpy(s->sym + 1, str);
+       s->sym = (unsigned char *) malloc(s->len + 1);
+       strcpy((char *)s->sym + 1, str);
        s->sym[0] = s->type;
 
        return 0;
@@ -197,16 +197,16 @@
                 * move then they may get dropped in pass 2, which breaks the
                 * symbols rules.
                 */
-               if (s->addr == _etext && strcmp(s->sym + offset, "_etext"))
+               if (s->addr == _etext && strcmp((char *)s->sym + offset, 
"_etext"))
                        return 0;
        }
 
        /* Exclude symbols which vary between passes. */
-       if (strstr(s->sym + offset, "_compiled."))
+       if (strstr((char *)s->sym + offset, "_compiled."))
                return 0;
 
        for (i = 0; special_symbols[i]; i++)
-               if( strcmp(s->sym + offset, special_symbols[i]) == 0 )
+               if( strcmp((char *)s->sym + offset, special_symbols[i]) == 0 )
                        return 0;
 
        return 1;

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog

<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-changelog] The attached patch adds -Werror to HOSTCFLAGS in Config.mk, makes, Xen patchbot -unstable <=