Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CMake+Rust: Fix offline builds w. vendored git repos #1120

Merged

Conversation

micahsnyder
Copy link
Contributor

Some builds using the tarball with the vendored Rust dependencies are failing.

The onenote-rs dependency is presently tied to a git branch from github rather than using a release from crates.io. This is the differing factor though I'm unsure why it is causing the build to try to update the repo rather than just building the vendored source.

This commit adds a --offline parameter to the build options if the vendored source is detected, in an attempt to force Cargo to use what it has and stay offline.

Some builds using the tarball with the vendored Rust dependencies are
failing.

The onenote-rs dependency is presently tied to a git branch from github
rather than using a release from crates.io. This is the differing factor
though I'm unsure why it is causing the build to try to update the repo
rather than just building the vendored source.

This commit adds a `--offline` parameter to the build options if the
vendored source is detected, in an attempt to force Cargo to use what it
has and stay offline.
@micahsnyder micahsnyder force-pushed the CLAM-2500-vendored-git-repos-offline branch 2 times, most recently from 13015db to ba435f8 Compare December 20, 2023 00:18
@micahsnyder micahsnyder force-pushed the CLAM-2500-vendored-git-repos-offline branch from ba435f8 to 4d2d890 Compare December 20, 2023 20:49
@micahsnyder micahsnyder merged commit c48146a into Cisco-Talos:main Jan 12, 2024
23 of 24 checks passed
@micahsnyder micahsnyder deleted the CLAM-2500-vendored-git-repos-offline branch January 12, 2024 16:33
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