Le 21/11/2015 04:19, Scott Kostyshak a écrit :
In the output of configure, I see:

...
checking whether NLS is requested... yes
checking for msgfmt... /usr/bin/msgfmt
checking for gmsgfmt... /usr/bin/msgfmt
checking for xgettext... /usr/bin/xgettext
checking for msgmerge... /usr/bin/msgmerge
checking whether NLS is requested... yes
...

It seems we have two checks for NLS. The attached patch attempts to fix
this. I have no idea what I'm doing here. I just observe that before the
patch the line is duplicated and after the patch there is only one
check.

You patch is correct. AM_PO_SUBDIRS already requires AM_NLS. Instead of removing the call, I would prefer to keep it and call AM_PO_SUBDIRS later, so that it does not need to invoke AM_NLS.

OK?

JMarc

>From 952701eb68a0666f9fa756e09c80f73cc7a800f4 Mon Sep 17 00:00:00 2001
From: Jean-Marc Lasgouttes <lasgout...@lyx.org>
Date: Mon, 23 Nov 2015 11:53:46 +0100
Subject: [PATCH] Do not invoke AM_NLS twice.

---
 configure.ac |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index f62a8d3..d82fcaa 100644
--- a/configure.ac
+++ b/configure.ac
@@ -170,13 +170,13 @@ if echo $prefix |grep ' ' >/dev/null 2>/dev/null ; then
 fi
 
 ### Setup po directory
-AM_PO_SUBDIRS
 AM_NLS
 if test $USE_NLS = "yes" ; then
   AC_DEFINE(ENABLE_NLS, 1,
     [Define to 1 if translation of program messages to the user's native language
    is requested.])dnl'
 fi
+AM_PO_SUBDIRS
 
 # some standard header files
 AC_HEADER_MAJOR
-- 
1.7.9.5

Reply via email to