* configure.ac: It seems installing in non-ASCII is not, in fact, ok.

diff --git a/ChangeLog b/ChangeLog
index c4fc0b2d04552c38d0b4f0bf1b283664435f60c3..da6fb6bb9740fe9f48f341c166f6724a79123049 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2013-10-27  Glenn Morris  <rgm@gnu.org>
+
+       * configure.ac: It seems installing in non-ASCII is not, in fact, ok.
+
 2013-10-25  Glenn Morris  <rgm@gnu.org>
 
        * configure.ac: It seems _installing_ in non-ASCII is ok, not building.

diff --git a/configure.ac b/configure.ac
index 2c1123f3a0b1e04e1c6aac07dedcf922c268a748..9263d3952449d7f21429aee73a725163befa99f1 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -74,6 +74,11 @@ dnl --program-transform-name options
 AC_ARG_PROGRAM
 
 dnl http://debbugs.gnu.org/15260
+dnl I think we have to check, eg, both exec_prefix and bindir,
+dnl because the latter by default is not yet expanded, but the user
+dnl may have specified a value for it via --bindir.
+dnl At first glance, _installing_ in non-ASCII seems ok, but in fact
+dnl it is not; see http://debbugs.gnu.org/15260#61
 dnl Note that abs_srcdir and abs_builddir are not yet defined. :(
 
 dnl "`cd \"$srcdir\"`" is not portable.
@@ -82,11 +87,12 @@ dnl "There is just no portable way to use double-quoted strings inside
 dnl double-quoted back-quoted expressions (pfew!)."
 temp_srcdir=`cd "$srcdir"; pwd`
 
-for var in "`pwd`" "$temp_srcdir"; do
+for var in "`pwd`" "$temp_srcdir" "$prefix" "$exec_prefix" \
+    "$datarootdir" "$bindir" "$datadir" "$sharedstatedir" "$libexecdir"; do
 
   dnl configure sets LC_ALL=C early on, so this range should work.
   case "$var" in
-    *[[^\ -~]]*) AC_MSG_ERROR([Emacs cannot be built in a directory whose name contains non-ASCII characters: $var]) ;;
+    *[[^\ -~]]*) AC_MSG_ERROR([Emacs cannot be built or installed in a directory whose name contains non-ASCII characters: $var]) ;;
   esac
 
 done