Re: rhel8 test failure confirmation?

Zack Weinberg wrote:
> On Mon, Dec 4, 2023, at 7:14 PM, Jacob Bachmeyer wrote:
>   
>> Zack Weinberg wrote:
>>     
> [snip everything addressed in the other thread]
>   
>> Yes, there was a bit of confusion here; not only is the FileUtils
>> module synchronized between autom4te and automake
>>     
>
> Thanks for reminding me that I need to make sure all those files are
> actually in sync before I cut the final 2.72 release.
>
>   
>>>       require Time::HiRes;
>>>       import Time::HiRes qw(stat);
>>>       
>> I believe that the import is not actually necessary
>>     
>
> The previous line is a "require", not a "use", so I believe it _is_
> necessary.  Have I misunderstood?
>
>   
>> ... should do no harm as long as any use of stat in the code
>> is prepared to handle floating-point timestamps.
>>     
>
> There's only one use, in 'sub mtime', and that's the place
> where we actively want the floating-point timestamps.

Yes, before seeing your actual patch, I had the mistaken impression that 
this code was in autom4te itself, not the FileUtils module.  The import 
is needed in the FileUtils module, so the patch is correct.


-- Jacob