Re: argp-tests: needs EXEEXT

[Simon Josefsson]

Simon Josefsson <address@hidden> writes:

> This patch fixes the argp failure in:
>
> http://autobuild.josefsson.org/gnulib/log-200810270833778763000.txt
>
> I have pushed it.
>
> I have been using the same idiom in e.g. gnutls for scripts that needs
> to invoke built programs.  Are there any thoughts on better ways to
> solve this problem?

I noticed that some test modules set EXEEXT in the module Makefile.am
section, and using the same idiom seems more consistent, so I have
pushed this.

/Simon

>From fb150740d1e516cf8fdd1effe7b1f6556471b418 Mon Sep 17 00:00:00 2001
From: Simon Josefsson <address@hidden>
Date: Tue, 28 Oct 2008 11:04:35 +0100
Subject: [PATCH] argp: Use standard gnulib idiom for EXEEXT.

---
 ChangeLog          |    6 ++++++
 gnulib-tool        |    2 +-
 modules/argp-tests |    1 +
 3 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 606e3a8..2b3c23b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2008-10-28  Simon Josefsson  <address@hidden>
+
+       * gnulib-tool (func_emit_tests_Makefile_am): Revert last commit.
+       * modules/argp-tests (test_argp_LDADD): Set EXEEXT here instead,
+       using idiom used elsewhere in gnulib.
+
 2008-10-27  Jim Meyering  <address@hidden>
 
        * modules/gethostname (License): Relicense: LGPL -> LGPLv2+.
diff --git a/gnulib-tool b/gnulib-tool
index c6d9dc1..b0f4c18 100755
--- a/gnulib-tool
+++ b/gnulib-tool
@@ -2012,7 +2012,7 @@ func_emit_tests_Makefile_am ()
   #   "AM_GNU_GETTEXT used but SUBDIRS not defined"
   echo "SUBDIRS ="
   echo "TESTS ="
-  echo "TESTS_ENVIRONMENT = EXEEXT=\$(EXEEXT)"
+  echo "TESTS_ENVIRONMENT ="
   echo "noinst_PROGRAMS ="
   if ! $for_test; then
     echo "check_PROGRAMS ="
diff --git a/modules/argp-tests b/modules/argp-tests
index 8f92a4d..8879cce 100644
--- a/modules/argp-tests
+++ b/modules/argp-tests
@@ -7,5 +7,6 @@ progname
 
 Makefile.am:
 TESTS += test-argp test-argp-2.sh
+TESTS_ENVIRONMENT += EXEEXT='@EXEEXT@'
 check_PROGRAMS += test-argp
 test_argp_LDADD = $(LDADD) @LIBINTL@
-- 
1.5.6.5