automake-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: getting rid of DEP_FILES


From: Tom Tromey
Subject: Re: getting rid of DEP_FILES
Date: 01 Jan 2004 14:42:30 -0700
User-agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.3.50

>>>>> "adl" == Alexandre Duret-Lutz <address@hidden> writes:

Thomas> - Tom suggested that DEP_FILES may be redundant.  He suggested that
Thomas> configure read the list of .Plo files from the am__include lines in
Thomas> Makefile.in.

adl> Is this close to what you had in mind?

Yes, thanks for implementing this.

BTW, what do you think of the appended?  This fixes a bug that Tom
Fitzsimmons noticed in the no-dist option.

Tom

Index: ChangeLog
from  Tom Tromey  <address@hidden>

        * automake.in (handle_source_transform): Don't generate dist
        variables when no-dist is set.
        (generate_makefile): Likewise.

Index: automake.in
===================================================================
RCS file: /cvs/automake/automake/automake.in,v
retrieving revision 1.1532
diff -u -r1.1532 automake.in
--- automake.in 1 Jan 2004 18:54:20 -0000 1.1532
+++ automake.in 1 Jan 2004 21:49:05 -0000
@@ -1829,7 +1829,7 @@
          unless $prefix =~ /EXTRA_/;
 
        push @sources, "\$($varname)";
-       if ($prefix !~ /^nodist_/)
+       if (! option 'no-dist' && $prefix !~ /^nodist_/)
          {
            # If the VAR wasn't defined conditionally, we add
            # it to DIST_SOURCES as is.  Otherwise we create a
@@ -6904,7 +6904,10 @@
 
   # Variables used by distdir.am and tags.am.
   define_pretty_variable ('SOURCES', TRUE, INTERNAL, @sources);
-  define_pretty_variable ('DIST_SOURCES', TRUE, INTERNAL, @dist_sources);
+  if (option 'no-dist')
+    {
+      define_pretty_variable ('DIST_SOURCES', TRUE, INTERNAL, @dist_sources);
+    }
 
   handle_multilib;
   handle_texinfo;




reply via email to

[Prev in Thread] Current Thread [Next in Thread]