[BUG] Infinite make recursion when configure.ac changes

2013-02-20 Thread Stefano Lattarini
From a pristine master checkout: $ make configure ./configure make ... # All successfull $ touch configure.ac $ make GEN config.status make[1]: Entering directory `/storage/home/stefano/git/src' GEN config.status make[2]: Entering directory `/storage/home/stefano/git/src'

Re: [BUG] Infinite make recursion when configure.ac changes

2013-02-20 Thread Jeff King
On Wed, Feb 20, 2013 at 09:44:58AM +0100, Stefano Lattarini wrote: From a pristine master checkout: $ make configure ./configure make ... # All successfull $ touch configure.ac $ make GEN config.status make[1]: Entering directory `/storage/home/stefano/git/src' GEN

Re: [BUG] Infinite make recursion when configure.ac changes

2013-02-20 Thread Junio C Hamano
Jeff King p...@peff.net writes: I can easily replicate it here. This seems due to the change in commit v1.7.12.4-1-g1226504: the issue is still present there, but no longer is in the preceding commit 7e201053 (a.k.a. v1.7.12.4). I haven't investigated this any further for the moment.

Re: [BUG] Infinite make recursion when configure.ac changes

2013-02-20 Thread Jeff King
On Wed, Feb 20, 2013 at 10:10:42PM -0800, Junio C Hamano wrote: I noticed this while looking at the other autoconf patch yesterday, but I was otherwise occupied in the evening and did not pursue it further. Thanks for looking into it. Here's the patch with a commit message. I'm pretty sure

Re: [BUG] Infinite make recursion when configure.ac changes

2013-02-20 Thread Junio C Hamano
Jeff King p...@peff.net writes: Anyway, here is the patch to fix the loop. -- 8 -- Subject: [PATCH] Makefile: avoid infinite loop on configure.ac change If you are using autoconf and change the configure.ac, the Makefile will notice that config.status is older than configure.ac, and will