[libcxx-commits] [libcxx] [libc++][chrono] Adds the sys_info class. (PR #85619)

[Mark de Wever via libcxx-commits]

mordante wrote:

> Hi @mordante ,
> 
> > I think the error can be improved, but I disagree that this is not useful.

Ah thanks. I'm working on a patch to show the source location of the caller, which would make it a lot easier to spot the issue.

> Anyway, I tried to find which part of the test is failed and I found that it gets failed on `Antarctica/Syowa` zone. Also, this test has passed on the Aarch64/Ubuntu Linux targets.
> 
<snip>
> Hope it will help in your investigation of the problem.

Thanks I think it will. I'll check whether these rules have changes, in which case it might depend on the version of the time zone database used. But not only the version seems to matter; there seem to be configuration options too. 2024a on Debian Stable works, but Debian sid is not compatible. (That is a different story which I'm working on.)

For the short term I keep this test disabled and will investigate this further; I first want to improve the output of the error messages.

https://github.com/llvm/llvm-project/pull/85619