Re: POSIX incompatiblity ("\?" in regular expression) in AM_PATH_LISPDIR

[Alexandre Duret-Lutz]

>>> "Paul" == Paul Eggert <address@hidden> writes:

 Paul> AM_PATH_LISPDIR assumes GNU sed's \? extension to POSIX in regular
 Paul> expressions.  Here's a proposed (but untested) patch to make it more
 Paul> portable to straight POSIX implementations.

Thank you Paul!  Since I saw you hadn't generated the patch from
CVS, I installed it on HEAD and branch-1-8 as follows.  However
you should really feel free to FYI that sort of fixes.

2004-01-10  Paul Eggert  <address@hidden>

        * m4/lispdir.m4 (AM_PATH_LISPDIR): Don't use \? in sed regular
        expressions; it doesn't conform to POSIX.

Index: m4/lispdir.m4
===================================================================
RCS file: /cvs/automake/automake/m4/lispdir.m4,v
retrieving revision 1.23
diff -u -r1.23 lispdir.m4
--- m4/lispdir.m4       13 Nov 2003 19:39:06 -0000      1.23
+++ m4/lispdir.m4       10 Jan 2004 14:00:52 -0000
@@ -4,7 +4,7 @@
 ## Almost entirely rewritten by Alexandre Oliva
 ## ------------------------
 
-# Copyright (C) 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003
+# Copyright (C) 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004
 # Free Software Foundation, Inc.
 
 # This program is free software; you can redistribute it and/or modify
@@ -22,7 +22,7 @@
 # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
 # 02111-1307, USA.
 
-# serial 7
+# serial 8
 
 # AM_PATH_LISPDIR
 # ---------------
@@ -50,8 +50,8 @@
   AC_RUN_LOG([$EMACS -batch -q -eval '(while load-path (princ (concat (car 
load-path) "\n")) (setq load-path (cdr load-path)))' </dev/null >conftest.out])
        am_cv_lispdir=`sed -n \
        -e 's,/$,,' \
-       -e 
'/.*\/lib\/x\?emacs\/site-lisp$/{s,.*/lib/\(x\?emacs/site-lisp\)$,${libdir}/\1,;p;q;}'
 \
-       -e 
'/.*\/share\/x\?emacs\/site-lisp$/{s,.*/share/\(x\?emacs/site-lisp\),${datadir}/\1,;p;q;}'
 \
+       -e 
'/.*\/lib\/x*emacs\/site-lisp$/{s,.*/lib/\(x*emacs/site-lisp\)$,${libdir}/\1,;p;q;}'
 \
+       -e 
'/.*\/share\/x*emacs\/site-lisp$/{s,.*/share/\(x*emacs/site-lisp\),${datadir}/\1,;p;q;}'
 \
        conftest.out`
        rm conftest.out
      fi

-- 
Alexandre Duret-Lutz