-
Notifications
You must be signed in to change notification settings - Fork 151
feat: Add vendored support #411
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
Changes from all commits
36fccfd
65107f2
d5419cb
e01ad2a
6b95074
d1c4510
e85db9a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| src/generated/ diff=false | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| [package] | ||
| name = "tikv-client-proto-build" | ||
| version = "0.0.0" | ||
| publish = false | ||
|
|
||
| keywords = ["TiKV", "KV", "distributed-systems"] | ||
| license = "Apache-2.0" | ||
| authors = ["The TiKV Project Authors"] | ||
| repository = "https://github.com/tikv/client-rust" | ||
| description = "The Rust language implementation of TiKV client." | ||
| edition = "2021" | ||
|
|
||
| [dependencies] | ||
| glob = "0.3" | ||
| tonic-build = "0.9" | ||
| # Suppress doctest bug (https://stackoverflow.com/questions/66074003/how-to-turn-off-cargo-doc-test-and-compile-for-a-specific-module-in-rust) | ||
| tonic-disable-doctest = "0.1.0" |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,12 +1,14 @@ | ||
| // Copyright 2020 TiKV Project Authors. Licensed under Apache-2.0. | ||
|
|
||
| use tonic_disable_doctest::BuilderEx; | ||
| // Copyright 2023 TiKV Project Authors. Licensed under Apache-2.0. | ||
|
|
||
| fn main() { | ||
| use tonic_disable_doctest::BuilderEx; | ||
|
|
||
| tonic_build::configure() | ||
| .disable_doctests_for_types([".google.api.HttpRule"]) | ||
| .emit_rerun_if_changed(false) | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. By the way, it's interesting to know why the auto rerun is disabled.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
After this change, we are running codegen in a new bin instead of |
||
| .build_server(false) | ||
| .include_file("mod.rs") | ||
| .out_dir("src/generated") | ||
| .compile( | ||
| &glob::glob("proto/*.proto") | ||
| .unwrap() | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.