tags 11896 + patch
severity 11896 minor
close 11896
thanks

On 07/10/2012 01:18 PM, Adam Sampson wrote:
> Hi,
>
Hi Adam, thanks for the report and the patch.

> I've just built automake-1.12.2 and had the aclocal7.sh test fail. After
> a bit of investigation, it's because the test:
> 
> - creates a .m4 file;
> - calls aclocal;
> - calls automake;
> - sleeps for a bit;
> - calls aclocal again;
> - calls automake again;
> - checks that those two calls didn't actually write any files.
>   [then goes on to test some other stuff that works fine]
> 
> The problem is that aclocal decides whether to rewrite its output file
> by checking that all of the input files it's read are older than the
> existing output file. If aclocal takes less than a second to run in the
> sequence above, it's possible for its output file to have the same
> timestamp as the .m4 file -- and then the second call to aclocal will
> decide the output file needs rewriting, and the test fails.
>
Oops, I introduced this error in a previous commit (v1.12.1-46-g13dd512).

> The attached patch adds a $sleep after creating the .m4 file, so it'll
> definitely be older than the first aclocal's output.
>
I've applied the fix in your name (see attached patch), and pushed.

I'm thus closing this bug report.

Thanks,
  Stefano