* update_autogen: Find loaddefs targets rather than parsing lisp/Makefile.in

* lisp/Makefile.in: Comment.

diff --git a/admin/ChangeLog b/admin/ChangeLog
index fc58259443fa2d05e6ffb32e1fcfa247451f39cd..a20e93d38c2378a5d9eff6da70e657592a015168 100644 (file)
--- a/admin/ChangeLog
+++ b/admin/ChangeLog
@@ -1,3 +1,8 @@
+2014-06-30  Glenn Morris  <rgm@gnu.org>
+
+       * update_autogen: Find loaddefs targets rather than
+       parsing lisp/Makefile.in
+
 2014-06-29  Glenn Morris  <rgm@gnu.org>
 
        * update_autogen: Remove need to cd into/out of lisp/.

diff --git a/admin/update_autogen b/admin/update_autogen
index 90431fa63014b5ded3b96c3891f941ec28bcd42e..ef0edfef50c0f6b6c74488be218ed892e387714d 100755 (executable)
--- a/admin/update_autogen
+++ b/admin/update_autogen
@@ -322,15 +322,19 @@ EOF
 
 echo "Finding loaddef targets..."
 
-sed -n -e '/^AUTOGEN_VCS/,/^$/p' lisp/Makefile.in | \
-    sed -e '/AUTOGEN_VCS/d' -e '/^$/d' -e 's/\\//' \
-    >| $tempfile || die "sed error"
+find lisp -name '*.el' -exec grep '^;.*generated-autoload-file:' {} + | \
+    sed -e '/loaddefs\|esh-groups/d' -e 's|/[^/]*: "|/|' -e 's/"//g' \
+    >| $tempfile || die "Error finding targets"
 
 genfiles=
 
 while read genfile; do
 
-    genfile=lisp/$genfile
+    ## Or we can just use sort -u when making tempfile...
+    case " $genfiles " in
+        *" $genfile "*) continue ;;
+    esac
+
     [ -r $genfile ] || die "Unable to read $genfile"
 
     genfiles="$genfiles $genfile"

diff --git a/lisp/Makefile.in b/lisp/Makefile.in
index fefbd5cda56adcbf44cb8fa9c81543375505375c..45a964b0d70931d0df85986c4086c2ed391a81cc 100644 (file)
--- a/lisp/Makefile.in
+++ b/lisp/Makefile.in
@@ -71,7 +71,6 @@ AUTOGENEL = loaddefs.el \
        org/org-loaddefs.el
 
 # Versioned files that are the value of someone's `generated-autoload-file'.
-# Note that update_loaddefs parses this.
 AUTOGEN_VCS = \
        ps-print.el \
        obsolete/tpu-edt.el \