Re: [PATCH] Modernize, improve and/or extend tests `colon*.test.

[Stefano Lattarini]

At Sunday 08 August 2010, Ralf Wildenhues wrote:
> * Stefano Lattarini wrote on Thu, Aug 05, 2010 at 01:54:20PM CEST:
> > Ping on this?  It seems to me that most issues with this patch
> > had already been solved, so it's probably a pity to let it
> > bitrotting here...
> 
> Well, I'd prefer the extract_dependencies script approach
I like the idea, too, but as I said I'd prefer to leave for a later 
patch.
> which would be more correct and not the complicated perl script
> which wasn't as correct.
Why not as correct?

You said in a previous mail:
 """
   A shell function in defs.in 'extract_makefile_deps TARGET FILE'
   that contains a sed script similar to extract_makefile_var that
   does what you want, and respects backslash-newline?
 """
But why should that function preserve backslash-newline occurrences?
Is not this an internal detail due to automatical line-wrapping by
Automake, which we should therefore ignore?

> And the relaxed file distribution test.  ;-)
Done (I said that I agreed with you on this): I squashed in this 
change:

diff --git a/tests/colon5.test b/tests/colon5.test
index 26f10e4..c53910e 100755
--- a/tests/colon5.test
+++ b/tests/colon5.test
@@ -34,7 +34,7 @@ cat > Makefile.am <<'END'
 .PHONY: test
 test:
        case ' $(DIST_COMMON) ' in \
-         *' $(srcdir)/Makefile.dep '*) exit 0;; \
+         *' $(srcdir)/Makefile.dep '*|*' Makefile.dep '*) exit 0;; \
          *) exit 1;; \
        esac
 END

> Thanks, and sorry for the delay,
> Ralf

Thanks for the review,
   Stefano