Yo Hal!
On Sat, 01 Feb 2025 17:23:55 -0800
Hal Murray via devel <devel@ntpsec.org> wrote:
> Anybody understand this?
Likely not.
> While I was cleaning up the long long in struct timex tangle, I added
> a line to wscript to document the size of struct struct timeval
> sizeofs = [
> ("time.h", "struct timespec"),
> + ("sys/time.h", "struct timeval"),
> ("time.h", "time_t"),
> (None, "long"),
> ]
DOn't do that. Per the man page:
The <sys/time.h> header, included by <time.h>, defines various
structures related to time and timers. "
Basically you include <time.h>, then CC includes <sys/time.h>, and
<sys/time.h> defines timespec AND timeval.
> That list gets sorted. (I don't know why.)
I like things sorted. Does not, usually, affect the code. But I don't
know that dictionary.
> But now it's dying on code that previously worked. I think.
Then why change the include test? The man page says the old one is
sufficient.
> Checking for field tai in struct ntptimeval : yes
> Checking sizeof long : 4
> Checking sizeof struct timespec (time.h) :
> Traceback (most recent call last):
> File
> "/builds/NTPsec/ntpsec/.waf3-2.0.25-767522112be77f8585812fcfaa08e805
> /waflib/Scripting.py", line 119, in waf_entry_point run_commands()
> File
> "/builds/NTPsec/ntpsec/.waf3-2.0.25-767522112be77f8585812fcfaa08e805
> /waflib/Scripting.py", line 182, in run_commands ...
>
> Details in https://gitlab.com/NTPsec/ntpsec/-/jobs/9016721971
What C code did that try to generate for the test?
RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
g...@rellim.com Tel:+1 541 382 8588
Veritas liberabit vos. -- Quid est veritas?
"If you can't measure it, you can't improve it." - Lord Kelvin
_______________________________________________
devel mailing list
devel@ntpsec.org
https://lists.ntpsec.org/mailman/listinfo/devel
