This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] for mig check in GDB's configure
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Thomas Schwinge <thomas at codesourcery dot com>
- Cc: Yue Lu <hacklu dot newborn at gmail dot com>, bug-hurd at gnu dot org, gdb-patches at sourceware dot org
- Date: Fri, 17 May 2013 09:30:42 +0400
- Subject: Re: [patch] for mig check in GDB's configure
- References: <CAB8fV=gfGtguD28FGa-A5DZT8jqvEA1AoaK4dO=cHMQcCVvB-w at mail dot gmail dot com> <8738u4sc19 dot fsf at kepler dot schwinge dot homeip dot net> <CAB8fV=hrvCmRd1AkMwvFsYZq1tC=7AgExWEfHLAURu6iWo3pTQ at mail dot gmail dot com> <87txmkxlu6 dot fsf at violet dot siamics dot net> <CAB8fV=j=sdO6DRTxBe8JxQ0kpSQxrL41y=zJ6EEOVP7xHXY+tA at mail dot gmail dot com> <871u96mvy7 dot fsf at kepler dot schwinge dot homeip dot net>
Hi Thomas and Yue,
> After reviewing and testing it, I now checked in your patch (very minor
> coding style and wording changes). Congratulations to your first
> contribution! :-)
>
> 2013-05-16 Yue Lu <hacklu.newborn@gmail.com>
>
> * configure.ac: Ensure MIG is available when building for GNU Hurd
> hosts.
> * configure: Regenerate.
I cannot remember if we said Yue had a copyright assignment on file,
or not. I checked the file, and couldn't locate him. Can you clarify?
This patch is small enough that it can be accepted without regardless;
but if we do, I'd like to mark it as such in the ChangeLog entry
(using "(tiny change)", as mentioned by the guide for GNU maintainers).
Thank you!
>
> Index: configure.ac
> ===================================================================
> RCS file: /cvs/src/src/gdb/configure.ac,v
> retrieving revision 1.203
> diff -u -p -r1.203 configure.ac
> --- configure.ac 10 May 2013 16:10:40 -0000 1.203
> +++ configure.ac 16 May 2013 21:50:18 -0000
> @@ -486,8 +486,15 @@ AC_CHECK_TOOL(AR, ar)
> AC_CHECK_TOOL(DLLTOOL, dlltool)
> AC_CHECK_TOOL(WINDRES, windres)
>
> -# Needed for GNU/Hurd.
> -AC_CHECK_TOOL(MIG, mig)
> +case $host_os in
> + gnu*)
> + # Needed for GNU Hurd hosts.
> + AC_CHECK_TOOL(MIG, mig)
> + if test x"$MIG" = x; then
> + AC_MSG_ERROR([MIG not found but required for $host hosts])
> + fi
> + ;;
> +esac
>
> # ---------------------- #
> # Checks for libraries. #
> Index: configure
> ===================================================================
> RCS file: /cvs/src/src/gdb/configure,v
> retrieving revision 1.391
> diff -u -p -r1.391 configure
> --- configure 10 May 2013 16:10:40 -0000 1.391
> +++ configure 16 May 2013 21:50:20 -0000
> [...]
>
>
> Grüße,
> Thomas
--
Joel