[PATCH] time::chrono: allow /etc/localtime to be a file

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch

[PATCH] time::chrono: allow /etc/localtime to be a file

Antero Mejr <antero@mailbox.org>

Details

Message ID: <20220707164627.6210-1-antero@mailbox.org>
DKIM signature: missing

Download raw message

2 years ago

Patch: +1 -1

/etc/localtime is often a symlink, but may be a TZif file, see tzset(3).
Parse the TZif when /etc/localtime is a file.

Signed-off-by: Antero Mejr <antero@mailbox.org>
---
 time/chrono/timezone.ha | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/time/chrono/timezone.ha b/time/chrono/timezone.ha
index 52615b35..534338d2 100644
--- a/time/chrono/timezone.ha
+++ b/time/chrono/timezone.ha
@@ -229,7 +229,7 @@ let TZ_LOCAL: timezone = timezone {
 		case let fp: str =>
 			yield fp;
 		case =>
-			return;
+			yield LOCALTIME_PATH;
 		};
 
 		const file = match (os::open(filepath)) {
-- 
2.36.1

Drew DeVault <sir@cmpwn.com>

Details

Message ID: <CLOO6SJLZY1L.1XVIP92WSOOW8@taiga>
In-Reply-To: <20220707164627.6210-1-antero@mailbox.org> (view parent)
DKIM signature: missing

Download raw message

2 years ago

Thanks!

To git@git.sr.ht:~sircmpwn/hare
   f7584535..087d4f71  master -> master

~sircmpwn/hare-dev

[PATCH] time::chrono: allow /etc/localtime to be a file