Re: tests updates

2010-11-04 Thread Stefano Lattarini
On Wednesday 03 November 2010, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Wed, Nov 03, 2010 at 02:27:47PM CET: On Monday 01 November 2010, Ralf Wildenhues wrote: I'm so totally behind on patches and not getting better, that the strategy of ignoring testsuite work will not help

Re: tests updates

2010-11-04 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Thu, Nov 04, 2010 at 01:03:17PM CET: On Wednesday 03 November 2010, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Wed, Nov 03, 2010 at 02:27:47PM CET: Just one question: what about the already-existing tests-init branch? Should I try to bring it to a

[PATCH] {maint} Document in detail some limitations of aclocal. (was: Re: Strangeness with m4_include and aclocal.)

2010-11-04 Thread Stefano Lattarini
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

Re: [PATCH v2] Overhauled and modularized tests in `instspc.test'.

2010-11-04 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Wed, Nov 03, 2010 at 08:47:54PM CET: On Wednesday 03 November 2010, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Wed, Nov 03, 2010 at 03:10:30PM CET: Pinging the patch again, following this:

Re: More problems with `make -n' in automake-generated rules.

2010-11-04 Thread Ralf Wildenhues
* Stefano Lattarini wrote on Wed, Nov 03, 2010 at 06:48:16PM CET: Hello Ralf. Again, just a couple of nits w.r.t. the test cases... Thanks; but I didn't mean to actually commit the second patch (just in case that wasn't clear). That said, I'll reply to your comments inline. On Monday 01

Re: More problems with `make -n' in automake-generated rules.

2010-11-04 Thread Stefano Lattarini
On Thursday 04 November 2010, Ralf Wildenhues wrote: Hi Stefano, * Stefano Lattarini wrote on Wed, Nov 03, 2010 at 06:30:34PM CET: On Monday 01 November 2010, Ralf Wildenhues wrote: I noticed more issues with automake-generated rules and `make -n': 1) The solutions documented in

Re: [PATCH] aclocal: handle ACLOCAL_PATH environment variable

2010-11-04 Thread Stefano Lattarini
On Thursday 04 November 2010, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Wed, Nov 03, 2010 at 04:24:51PM CET: On Tuesday 02 November 2010, Paolo Bonzini wrote: + # Add any directory listed in the `ACLOCAL_PATH' environment + # variable. + if (defined $ENV{ACLOCAL_PATH})

Re: More problems with `make -n' in automake-generated rules.

2010-11-04 Thread Stefano Lattarini
On Thursday 04 November 2010, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Wed, Nov 03, 2010 at 06:48:16PM CET: Hello Ralf. Again, just a couple of nits w.r.t. the test cases... Thanks; but I didn't mean to actually commit the second patch (just in case that wasn't clear). I didn't

Re: More problems with `make -n' in automake-generated rules.

2010-11-04 Thread Stefano Lattarini
On Thursday 04 November 2010, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Thu, Nov 04, 2010 at 09:50:08PM CET: On Thursday 04 November 2010, Ralf Wildenhues wrote: * Stefano Lattarini wrote on Wed, Nov 03, 2010 at 06:48:16PM CET: Hello Ralf. Again, just a couple of nits w.r.t.

Re: [PATCH] {maint} Document in detail some limitations of aclocal. (was: Re: Strangeness with m4_include and aclocal.)

2010-11-04 Thread Nick Bowler
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

Re: [PATCH] {maint} Document in detail some limitations of aclocal.

2010-11-04 Thread Nick Bowler
On 2010-11-04 22:28 +0100, Stefano Lattarini wrote: On Thursday 04 November 2010, Nick Bowler wrote: On 2010-11-04 20:47 +0100, Stefano Lattarini wrote: +AC_DEFUN([MY_DEFUN], [m4_apply([AC_DEFUN], [$1], [$2])]) This is insufficiently quoted, it should be:

Re: [PATCH] {maint} Document in detail some limitations of aclocal.

2010-11-04 Thread Ralf Wildenhues
Hello, * Stefano Lattarini wrote on Thu, Nov 04, 2010 at 08:47:47PM CET: 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? There are a couple of things I don't like with the principle