Skip to content

Fix panic when vendored protoc is unavailable#85

Merged
imLinguin merged 1 commit intoimLinguin:mainfrom
AOSC-Tracking:aosc/v0.3.2
Jan 30, 2026
Merged

Fix panic when vendored protoc is unavailable#85
imLinguin merged 1 commit intoimLinguin:mainfrom
AOSC-Tracking:aosc/v0.3.2

Conversation

@SkyBird233
Copy link
Contributor

Replace unwrap() with fallback to system protoc when protoc-bin-vendored fails, preventing build panics on unsupported platforms (e.g. LoongArch).

Copy link
Owner

@imLinguin imLinguin left a comment

Choose a reason for hiding this comment

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

Thanks, please run cargo fmt and it'll be ready to merge

@SkyBird233
Copy link
Contributor Author

SkyBird233 commented Jan 30, 2026

Sorry, let me try again with updated rustfmt

Replace unwrap() with fallback to system protoc when protoc-bin-vendored
fails, preventing build panics on unsupported platforms (e.g. LoongArch).
@SkyBird233 SkyBird233 requested a review from imLinguin January 30, 2026 09:56
@imLinguin imLinguin merged commit 15b9261 into imLinguin:main Jan 30, 2026
1 check passed
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