From: Paul Eggert <eggert@cs.ucla.edu>
Date: Fri, 7 Oct 2011 20:59:10 +0000 (-0700)
Subject: gnulib-tool: don't follow dependencies to avoided modules
X-Git-Tag: v0.1~1673
X-Git-Url: http://erislabs.org.uk/gitweb/?a=commitdiff_plain;h=4f11d6bebc3098c64ffde27079ab0d0cecfd0cdc;p=gnulib.git

gnulib-tool: don't follow dependencies to avoided modules

This fixes a bug that is related to the previous one.
* gnulib-tool (func_modules_transitive_closure)
(func_emit_autoconf_snippets):
Check whether a dependency is acceptable before using it.
(--extract-dependencies): Report an error if --avoid is also used,
since this combination of options is not yet supported.
---

diff --git a/ChangeLog b/ChangeLog
index c57a4fb42..e48cc62e9 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,13 @@
 2011-10-07  Paul Eggert  <eggert@cs.ucla.edu>
 
+	gnulib-tool: don't follow dependencies to avoided modules
+	This fixes a bug that is related to the previous one.
+	* gnulib-tool (func_modules_transitive_closure)
+	(func_emit_autoconf_snippets):
+	Check whether a dependency is acceptable before using it.
+	(--extract-dependencies): Report an error if --avoid is also used,
+	since this combination of options is not yet supported.
+
 	gnulib-tool: fix typo that broke Emacs on powerpc-apple-darwin9.8.0.
 	Problem reported by Peter Dyballa in
 	<http://debbugs.gnu.org/cgi/bugreport.cgi?bug=9696>.
diff --git a/gnulib-tool b/gnulib-tool
index 628d9d655..2e3b6cad3 100755
--- a/gnulib-tool
+++ b/gnulib-tool
@@ -2731,6 +2731,9 @@ func_modules_transitive_closure ()
               esac
             done
             if $inc; then
+              func_acceptable $dep || inc=false
+            fi
+            if $inc; then
               func_append inmodules " $dep"
               if test "$cond_dependencies" = true; then
                 escaped_dep=`echo "$dep" | sed -e "$sed_escape_dependency"`
@@ -3958,7 +3961,9 @@ func_emit_autoconf_snippets ()
           echo "      $shellvar=true"
           deps=`func_get_dependencies $module | sed -e "$sed_dependencies_without_conditions"`
           for dep in $deps; do
-            if func_cond_module_p "$dep"; then
+            if ! func_acceptable $dep; then
+              :
+            elif func_cond_module_p "$dep"; then
               func_module_shellfunc_name "$dep"
               func_cond_module_condition "$module" "$dep"
               if test "$condition" != true; then
@@ -3988,7 +3993,9 @@ func_emit_autoconf_snippets ()
         else
           deps=`func_get_dependencies $module | sed -e "$sed_dependencies_without_conditions"`
           for dep in $deps; do
-            if func_cond_module_p "$dep"; then
+            if ! func_acceptable $dep; then
+              :
+            elif func_cond_module_p "$dep"; then
               func_module_shellfunc_name "$dep"
               func_cond_module_condition "$module" "$dep"
               if test "$condition" != true; then
@@ -6507,6 +6514,9 @@ s/\([.*$]\)/[\1]/g'
     ;;
 
   extract-dependencies )
+    if test -n "$avoidlist"; then
+      func_fatal_error "cannot combine --avoid and --extract-dependencies"
+    fi
     for module
     do
       func_verify_module