Re: CHARSETALIASDIR

[Ben Pfaff]

John Darrington <address@hidden> writes:

> On Wed, Nov 14, 2007 at 09:32:28PM -0800, Ben Pfaff wrote:
>       I'm proposing to add the line
>              TESTS_ENVIRONMENT += CHARSETALIASDIR='$(abs_top_builddir)/gl'
>      to tests/automake.mk.  Does this sound reasonable?
>
> Perfectly.  But please also add a short comment explaining what it's
> for. 

Thanks.  I checked this in:

diff -u -p -r1.117 ChangeLog
--- tests/ChangeLog     10 Nov 2007 21:28:04 -0000      1.117
+++ tests/ChangeLog     15 Nov 2007 15:00:25 -0000
@@ -1,3 +1,9 @@
+2007-11-15  Ben Pfaff  <address@hidden>
+
+       * automake.mk: Add definition of CHARSETALIASDIR to
+       TESTS_ENVIRONMENT, to allow locale_charset to find charset.alias
+       before running "make install".
+
 2007-11-10  Ben Pfaff  <address@hidden>
 
        * bugs/compression.sh: Pass -b option to diff to avoid spurious
diff -u -p -r1.46 automake.mk
--- tests/automake.mk   9 Nov 2007 03:06:30 -0000       1.46
+++ tests/automake.mk   15 Nov 2007 15:00:25 -0000
@@ -2,6 +2,10 @@
 
 TESTS_ENVIRONMENT = top_srcdir='$(top_srcdir)' top_builddir='$(top_builddir)'
 TESTS_ENVIRONMENT += PERL='@PERL@'
+
+# Allow locale_charset to find charset.alias before running "make install".
+TESTS_ENVIRONMENT += CHARSETALIASDIR='$(abs_top_builddir)/gl'
+
 dist_TESTS = \
        tests/command/aggregate.sh \
        tests/command/autorecod.sh \

-- 
"Premature optimization is the root of all evil."
--D. E. Knuth, "Structured Programming with go to Statements"