Skip to content

Fix build in Windows C Runtime - #396

Merged
evanmiller merged 1 commit into
WizardMac:devfrom
hpoettker:fix-build
Sep 20, 2026
Merged

evanmiller merged 1 commit into
WizardMac:devfrom
hpoettker:fix-build

Conversation

@hpoettker

Copy link
Copy Markdown
Contributor

In the Windows C Runtime, the POSIX function localtime_r, which is a thread-safe alternative to localtime, is not available.

It offers the function localtime_s instead, which differs from the C11 function of that name in both parameter order and return value.

@hpoettker
hpoettker force-pushed the fix-build branch 2 times, most recently from 5ab7b14 to 8ab104b Compare September 20, 2026 19:21
In the Windows C Runtime, the POSIX function `localtime_r`, which
is a thread-safe alternative to `localtime`, is not available.

It offers the function `localtime_s` instead, which differs from
the C11 function of that name in both parameter order and return
value.
@hpoettker

Copy link
Copy Markdown
Contributor Author

Sorry for the multiple pushes.

The AppVeyor check is now successful as intended. In the commit amendments I tried to fix the artifact in the logs of the check that highlights an intended write to stderr of ReadStat_Test_sav_date. But that went nowhere and all changes to appveyor.yml are now reverted. The current commit contains the same changes as the first one.

@evanmiller
evanmiller merged commit 835b88c into WizardMac:dev Sep 20, 2026
12 checks passed
@hpoettker
hpoettker deleted the fix-build branch September 20, 2026 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants