automake-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#61240] [PATCH 2/2] Gracefully degrade if Time::HiRes is not availab


From: Jacob Bachmeyer
Subject: [bug#61240] [PATCH 2/2] Gracefully degrade if Time::HiRes is not available
Date: Tue, 07 Feb 2023 23:29:30 -0600
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.8.1.22) Gecko/20090807 SeaMonkey/1.1.17 Mnenhy/0.7.6.0

Paul Eggert wrote:
On 2023-02-05 21:43, Jacob Bachmeyer wrote:
Should the patch be relative to commit 6d6fc91c472fd84bd71a1b012fa9ab77bd94efea (before the version requirement bump) or should it include reverting commit 4e3744a15c4d8bdb46c11ead2fb56c5f591b714b (the version requirement bump)?

Might as well do it all at once, thanks.

This second patch is the change I previously suggested. As noted in the patch, please use my gnu.org address for public attribution. (I would prefer to minimize any indirect advertising of GMail.)

I have found no better ChangeLog entry than the commit message:

* lib/Automake/FileUtils.pm: Gracefully degrade if Time::HiRes is not available.

8<------
From: Jacob Bachmeyer <jcb@gnu.org>
Date: Tue, 7 Feb 2023 22:42:59 -0600
Subject: [PATCH 2/2] Gracefully degrade if Time::HiRes is not available

---
lib/Automake/FileUtils.pm |    5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/lib/Automake/FileUtils.pm b/lib/Automake/FileUtils.pm
index 697ff7e..8d0b368 100644
--- a/lib/Automake/FileUtils.pm
+++ b/lib/Automake/FileUtils.pm
@@ -39,9 +39,12 @@ use strict;
use warnings FATAL => 'all';

use Exporter;
-use Time::HiRes qw(stat);
use IO::File;

+# use sub-second resolution timestamps if available,
+# carry on with one-second resolution timestamps if that is all we have
+BEGIN { eval { require Time::HiRes; import Time::HiRes qw(stat) } }
+
use Automake::Channels;
use Automake::ChannelDefs;

--
8<------


-- Jacob






reply via email to

[Prev in Thread] Current Thread [Next in Thread]