Skip to content

boring-sys: Support static MSVC runtime#475

Merged
kornelski merged 1 commit intocloudflare:masterfrom
signalapp:msvc-crt-static
Feb 13, 2026
Merged

boring-sys: Support static MSVC runtime#475
kornelski merged 1 commit intocloudflare:masterfrom
signalapp:msvc-crt-static

Conversation

@jrose-signal
Copy link
Contributor

Adjustment to c299b14 to support +crt-static builds.

(I don't consider myself a Windows expert; perhaps there's a better way to deal with the original issue. But this should be a strict improvement.)

let unix = env::var("CARGO_CFG_UNIX").is_ok();

let target_features = env::var("CARGO_CFG_TARGET_FEATURE")
.unwrap()
Copy link
Collaborator

Choose a reason for hiding this comment

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

use unwrap_or_default.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Boo to Cargo for not being consistent :-P Thanks.

@kornelski kornelski merged commit ee95e20 into cloudflare:master Feb 13, 2026
24 checks passed
@jrose-signal jrose-signal deleted the msvc-crt-static branch February 13, 2026 02:00
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