-
Notifications
You must be signed in to change notification settings - Fork 486
Remove spam from output of cargo_build_script
s
#572
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
Comments
I believe this was fixed in 8c388e1 |
@UebelAndre: Great; thanks for the fix and for letting me know! (I can’t actually test it because we’ll have to upgrade past the |
wchargin
added a commit
to tensorflow/tensorboard
that referenced
this issue
Feb 4, 2021
Summary: We upgrade to latest `rules_rust` mainly for bazelbuild/rules_rust#572. This requires upgrading `cargo-raze` to the newly released v0.9.2, plus a herd of interdependent changes: - update to `rules_rust` latest - rename `@io_bazel_rules_rust` to `@rules_rust` - update Prost `protoc` wiring (not sure exactly why the old one doesn’t work anymore, but the new one is nicer, anyway) - update doc server to remove arch name, which also makes it work when compiled with `-c opt` Test Plan: Build changes only, so that `bazel test` passes and a `--load_fast` smoke screen looks good suffices for correctness. To see that the `openssl-sys` build spam is gone: ``` rm -rf bazel-out/k8-opt/bin/external/raze__openssl_sys__0_9_60 # ignore EACCES bazel build @raze__openssl_sys__0_9_60//:openssl_sys ``` wchargin-branch: rules-rust-20210203 wchargin-source: 05097a937b3ac58b873e3df69c89ddb887104a19
wchargin
added a commit
to tensorflow/tensorboard
that referenced
this issue
Feb 9, 2021
Summary: We upgrade to latest `rules_rust` mainly for bazelbuild/rules_rust#572. This requires upgrading `cargo-raze` to the newly released v0.9.2, plus a herd of interdependent changes: - update to `rules_rust` latest - rename `@io_bazel_rules_rust` to `@rules_rust` - update Prost `protoc` wiring (not sure exactly why the old one doesn’t work anymore, but the new one is nicer, anyway) - update doc server to remove arch name, which also makes it work when compiled with `-c opt` Test Plan: Build changes only, so that `bazel test` passes and a `--load_fast` smoke screen looks good suffices for correctness. To see that the `openssl-sys` build spam is gone: ``` rm -rf bazel-out/k8-opt/bin/external/raze__openssl_sys__0_9_60 # ignore EACCES bazel build @raze__openssl_sys__0_9_60//:openssl_sys ``` To test the doc server: ``` bazel run //tensorboard/data/server:rustboard_core_doc_server ``` …and navigate to the listed port. wchargin-branch: rules-rust-20210203
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
I’m transitively building
openssl-sys
viarules_rust
with buildfiles generated by
cargo-raze
. Building the crate generates a bunch oflog spam:
Best I can tell, these are from
openssl-sys
’s build script itself:When running with normal Cargo, there’s no such spam, so I think that
our
cargo_build_script_runner
should suppress these lines.edit: I’m at rules_rust-afee872f08ed67b53caab2e1d2fefd501dce81f4.
The text was updated successfully, but these errors were encountered: