From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24843 invoked by alias); 29 Apr 2013 17:05:35 -0000 Mailing-List: contact gdb-prs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-prs-owner@sourceware.org Received: (qmail 24808 invoked by uid 48); 29 Apr 2013 17:05:34 -0000 From: "tromey at redhat dot com" To: gdb-prs@sourceware.org Subject: [Bug build/15414] Build fails with mingw32-w64 GCC-4.8.0 Date: Mon, 29 Apr 2013 17:05:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gdb X-Bugzilla-Component: build X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: tromey at redhat dot com X-Bugzilla-Status: NEW X-Bugzilla-Priority: P2 X-Bugzilla-Assigned-To: unassigned at sourceware dot org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: X-Bugzilla-URL: http://sourceware.org/bugzilla/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 X-SW-Source: 2013-q2/txt/msg00168.txt.bz2 http://sourceware.org/bugzilla/show_bug.cgi?id=15414 --- Comment #3 from Tom Tromey 2013-04-29 17:05:34 UTC --- Ok, I see. Sorry about that, I misunderstood. I think the logic in configure.ac for setting this up is not correct: build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ -Wformat-nonliteral -Wpointer-sign \ -Wno-unused -Wunused-value -Wunused-function \ -Wno-switch -Wno-char-subscripts -Wmissing-prototypes \ -Wdeclaration-after-statement -Wempty-body" # Enable -Wno-format by default when using gcc on mingw since many # GCC versions complain about %I64. case "${host}" in *-*-mingw32*) build_warnings="$build_warnings -Wno-format" ;; esac It seems to me it would be better to try -Wformat only on other systems like: diff --git a/gdb/configure.ac b/gdb/configure.ac index bb7fbdd..7c2080c 100644 --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -1926,7 +1926,7 @@ fi # NOTE: If you change this list, remember to update # gdb/doc/gdbint.texinfo. build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ --Wformat-nonliteral -Wpointer-sign \ +-Wpointer-sign \ -Wno-unused -Wunused-value -Wunused-function \ -Wno-switch -Wno-char-subscripts -Wmissing-prototypes \ -Wdeclaration-after-statement -Wempty-body" @@ -1935,6 +1935,7 @@ build_warnings="-Wall -Wdeclaration-after-statement -Wpointer-arith \ # GCC versions complain about %I64. case "${host}" in *-*-mingw32*) build_warnings="$build_warnings -Wno-format" ;; + *) build_warnings="$build_warnings -Wformat-nonliteral" ;; esac AC_ARG_ENABLE(build-warnings, Could you give that a try? You should be able to apply this patch directly to configure if you can't re-run autoconf. -- Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.