Skip to content

fix: arm32 compilation#74

Merged
reneca merged 1 commit intomainfrom
arm32
Apr 27, 2026
Merged

fix: arm32 compilation#74
reneca merged 1 commit intomainfrom
arm32

Conversation

@reneca
Copy link
Copy Markdown
Contributor

@reneca reneca commented Apr 25, 2026

There is an issue for ARM32 where the time_t is a 32 bits instead of a 64 bits.
It lead to a compilation error.

This fix replace the i64 cast by a try_into and handle the error.

Signed-off-by: Jeremy HERGAULT <jeremy.hergault@worldline.com>
@reneca reneca requested review from Timmy80 and oschijns April 25, 2026 10:59
@reneca reneca self-assigned this Apr 25, 2026
@reneca reneca added the bug Something isn't working label Apr 25, 2026
Copy link
Copy Markdown
Contributor

@oschijns oschijns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This also fixes compilation for i686 targets.
I tested it with

cargo zigbuild --target i686-unknown-linux-gnu --features=openssl-vendored

And the generated cargo-prosa binary also run successfully on a x86_64 Linux OS.

@reneca reneca merged commit b318403 into main Apr 27, 2026
6 of 7 checks passed
@reneca reneca deleted the arm32 branch April 27, 2026 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants