* [RFA/RFC] disable-nls if msgfmt could not be found
@ 2003-03-18 18:33 Joel Brobecker
2003-03-18 19:16 ` Joseph S. Myers
0 siblings, 1 reply; 2+ messages in thread
From: Joel Brobecker @ 2003-03-18 18:33 UTC (permalink / raw)
To: gcc-patches, gdb-patches, binutils
[-- Attachment #1: Type: text/plain, Size: 1452 bytes --]
[I am resending this message that was only sent to gdb-patches@]
Hello,
Some users building GDB on systems where msgfmt is not available were
blocked when the build failed (in bfd, and probably another subdir
that I can't remember). See
http://sources.redhat.com/ml/gdb/2003-02/msg00310.html.
Instead of adding the checks in each subdir, I thought that it might be
better to do the check for msgfmt in the toplevel configure, and pass
--disable-nls to the subdir configure commands if not found. The
following patches tries to do this, and has been tested on a hpux box
where msgfmt is missing, and on a linux machine where msgfmt was
available.
Ideally, we want to do the check for msgfmt only if --disable-nls was
not already specified. However, the only consequence of not checking
for the presence of --disable-nls is that we might have more than one
--disable/enable-nls switches on the subdir configure commands. I
think it will be ok, since I've placed the --disable-nls at the end
so it should overide the user's switch. An AC_MSG_WARN may also have
been nice, but I decided against it because seeing a warning even
when the user has specified --disable-nls would seem strange...
Would the following change be acceptable for inclusion?
2003-03-17 J. Brobecker <brobecker@gnat.com>
* configure.in: Add check for msgfmt, configure subdirs with
--disable-nls if not found.
* configure: Regenerate.
--
Joel
[-- Attachment #2: configure.in.diff --]
[-- Type: text/plain, Size: 830 bytes --]
Index: configure.in
===================================================================
RCS file: /cvs/src/src/configure.in,v
retrieving revision 1.158
diff -c -3 -r1.158 configure.in
*** configure.in 12 Mar 2003 20:47:07 -0000 1.158
--- configure.in 18 Mar 2003 03:43:33 -0000
***************
*** 1824,1829 ****
--- 1824,1838 ----
*) gxx_include_dir=${with_gxx_include_dir} ;;
esac
+ # Check that msgfmt can be found in the PATH, or configure with
+ # --disable-nls to avoid a build failure.
+ AC_CHECK_PROG(MSGFMT, msgfmt, msgfmt, )
+ if [ test "x$MSGFMT" = "x" ]; then
+ build_configargs="${build_configargs} --disable-nls"
+ host_configargs="${host_configargs} --disable-nls"
+ target_configargs="${target_configargs} --disable-nls"
+ fi
+
FLAGS_FOR_TARGET=
case " $target_configdirs " in
*" newlib "*)
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [RFA/RFC] disable-nls if msgfmt could not be found
2003-03-18 18:33 [RFA/RFC] disable-nls if msgfmt could not be found Joel Brobecker
@ 2003-03-18 19:16 ` Joseph S. Myers
0 siblings, 0 replies; 2+ messages in thread
From: Joseph S. Myers @ 2003-03-18 19:16 UTC (permalink / raw)
To: Joel Brobecker; +Cc: gcc-patches, gdb-patches, binutils
On Tue, 18 Mar 2003, Joel Brobecker wrote:
> Instead of adding the checks in each subdir, I thought that it might be
> better to do the check for msgfmt in the toplevel configure, and pass
> --disable-nls to the subdir configure commands if not found. The
--disable-nls disables the *use* of compiled message catalogs, not just
their generation; GCC releases include the compiled message catalogs so
that users can get an NLS-enabled compiler whether or not they have msgfmt
installed.
--
Joseph S. Myers
jsm28@cam.ac.uk
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2003-03-18 19:16 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-03-18 18:33 [RFA/RFC] disable-nls if msgfmt could not be found Joel Brobecker
2003-03-18 19:16 ` Joseph S. Myers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).