On 2010-11-04 20:47 +0100, Stefano Lattarini wrote: > On Thursday 28 October 2010, Stefano Lattarini wrote: > > On Wednesday 27 October 2010, Nick Bowler wrote: > > > On 2010-10-23 15:23 +0200, Stefano Lattarini wrote: > > > > So I think your first problem is "just" an aclocal limitation we should > > > > resign to live with. But I also think that such a limitation should be > > > > documented explicitly, since it's by no means obvious. > > > > > > It's especially non-obvious because the other autotools handle this just > > > fine. > > Agreed. > I've gone ahead and documented this non-obvious limitation, and another > similar one regarding AC_DEFUN. See the attached patch. > > Ralf, ok to apply to maint? > > Nick, the attached patch also adds your name and e-mail address to the > automake THANKS file; do you have any problem with that?
No problem. I have a couple comments, however: > -*-*-*- > > Document in detail some limitations of aclocal. [...] > +...@example > +## Let's see an @command{aclocal} limitation w.r.t.@: scanning > +## of macro definitons. > +$ @kbd{cat m4/foo.m4} > +# Indirection used here, to avoid triggering the bug described > +# above. > +AC_DEFUN([MY_DEFUN], [m4_apply([AC_DEFUN], [$1], [$2])]) This is insufficiently quoted, it should be: [m4_apply([AC_DEFUN], [[$1], [$2]])] [...] > +These limitations might be lifted in a future version of > +...@command{aclocal}, but then again, they might not, so you > +should be preperated to live with them. ^^^^^^^^^^ prepared [...] > +++ b/tests/aclocal-limit-defun.test [...] > +cat > m4/foo.m4 <<'END' > +AC_DEFUN([MY_DEFUN], [m4_apply([AC_DEFUN], [$1], [$2])]) Same quoting issue as above. -- Nick Bowler, Elliptic Technologies (http://www.elliptictech.com/)