From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28575 invoked by alias); 3 May 2013 10:44:09 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 28562 invoked by uid 89); 3 May 2013 10:44:08 -0000 X-Spam-SWARE-Status: No, score=-4.1 required=5.0 tests=BAYES_00,FREEMAIL_FROM,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE,SPF_PASS autolearn=ham version=3.3.1 Received: from mail-lb0-f175.google.com (HELO mail-lb0-f175.google.com) (209.85.217.175) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Fri, 03 May 2013 10:44:07 +0000 Received: by mail-lb0-f175.google.com with SMTP id w20so1383028lbh.6 for ; Fri, 03 May 2013 03:44:04 -0700 (PDT) X-Received: by 10.112.157.102 with SMTP id wl6mr4045827lbb.65.1367577844519; Fri, 03 May 2013 03:44:04 -0700 (PDT) MIME-Version: 1.0 Received: by 10.152.123.80 with HTTP; Fri, 3 May 2013 03:43:49 -0700 (PDT) In-Reply-To: <8738u4sc19.fsf@kepler.schwinge.homeip.net> References: <8738u4sc19.fsf@kepler.schwinge.homeip.net> From: =?UTF-8?B?6ZmG5bKz?= Date: Fri, 03 May 2013 10:44:00 -0000 Message-ID: Subject: Re: [patch] for mig check in GDB's configure To: Thomas Schwinge Cc: bug-hurd@gnu.org, gdb-patches@sourceware.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-SW-Source: 2013-05/txt/msg00049.txt.bz2 Hi! thanks for your review. On Fri, May 3, 2013 at 4:28 PM, Thomas Schwinge w= rote: > > As GDB is a GNU project, instead of just a commit message it uses > ChangeLog files. See the several ChangeLog files in the GDB sources. As > your change only touches files in gdb/, only gdb/ChangeLog is relevant. > The format of the individual "snippets" is rather strict, see the > existing ones as well as this chapter in the GNU Coding Standards: > . I have modified the ChangeLog file under gdb/ . > >> --- a/gdb/configure >> +++ b/gdb/configure > > I take it you used autoconf to regenerate that file? Yes! I have already removed this. >> --- a/gdb/configure.ac >> +++ b/gdb/configure.ac >> @@ -488,6 +488,15 @@ AC_CHECK_TOOL(WINDRES, windres) >> >> # Needed for GNU/Hurd. >> AC_CHECK_TOOL(MIG, mig) >> +case "${host}" in > > Hmm, I think that instead of only examining the host system, $host, this > also needs to examine the target system, $target. (Please tell if the > difference between build, host, and target system is not clear to you.) > The MIG tool is used to generate files (from RPC definition files) that > are used by the native GDB port for GNU Hurd (which, of couse, is the > only GNU Hurd port that currently exists.) But if someone, for example, > builds GDB targeting mips-linux-gnu on a GNU Hurd system, they would not > need the MIG tool. > To my knowledge now, $target just need to set when building a compiler which specify which plateform your compiler generate code for. When we build GDB, it is trivial to check the variable. In your example, builds GDB targeting mips-linux-gnu means the GDB is running on mips-linux. So we only need to set the $host=3Dmips-linux-gnu, $build=3D*-*-gnu. Maybe I have got a wrong understanding about these gcc terms. > Can you change your patch according to my review and then resend it? > (Don't worry -- it is completely normal that patches are revised, even > several times, before they're approved. This helps to maintain a high > code quality.) > the new one is here: =46rom 13d3edd1f6dbbc20b2801cea1fc367bf9042f977 Mon Sep 17 00:00:00 2001 From: hacklu Date: Fri, 3 May 2013 18:27:08 +0800 Subject: [PATCH] Patch check mig on GNU Hurd 2013-05-3 hacklu * configure.ac : uncorrectly check for mig on GUN Hurd * configure: Regenerate. --- gdb/ChangeLog | 4 ++++ gdb/configure.ac | 9 +++++++++ 2 files changed, 13 insertions(+), 0 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 12254b7..015a878 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2013-05-3 hacklu + + * configure.ac : uncorrectly check for mig on GUN Hurd + * configure: Regenerate. 2013-04-30 Samuel Thibault * i386gnu-nat.c (CREG_OFFSET): New macro. diff --git a/gdb/configure.ac b/gdb/configure.ac index bb7fbdd..c1ee4cb 100644 --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -488,6 +488,15 @@ AC_CHECK_TOOL(WINDRES, windres) # Needed for GNU/Hurd. AC_CHECK_TOOL(MIG, mig) +case "${host}" in + *-linux*|*-k*bsd-gnu*) + ;; + i[?]86-*-gnu*) + if test "$MIG" =3D "" ; then + AC_MSG_ERROR([MIG not found but required for $host]) + fi + ;; +esac # ---------------------- # # Checks for libraries. # -- 1.7.0.4 --=20 Yue Lu (=E9=99=86=E5=B2=B3)