crate_universe/src/resolver.rs (8 lines): - line 46: // TODO: Interesting edge cases - line 82: // TODO: Ignore * .cargo config files outside of the workspace - line 85: // TODO: Combine values better - line 148: // TODO: improve the caching by generating a lockfile over the resolve rather than over - line 226: // TODO: Include all files referenced by the toml. - line 283: // TODO: These are ?all ignored - line 318: // TODO: generate a cargo toml from metadata in the bazel rule, when no cargo toml is present. - line 424: "TODO: cargo --version failed with exit code {:?}", rust/private/rustc.bzl (6 lines): - line 229: # TODO: We could let the user choose how to link, instead of always preferring to link static libraries. - line 840: # TODO: Remove after some resolution to - line 890: # TODO: Remove after some resolution to - line 936: # TODO - What to do for no_std? - line 949: # TODO: if we already have an rlib in our deps, we could skip this - line 1118: # TODO: Consider switching dylibs to use -Clink-arg as above. crate_universe/src/parser.rs (5 lines): - line 84: // workspace: TODO: support cargo workspaces. - line 85: // target: TODO: support platform-specific dependencies. - line 212: // TODO: only mark packages as conflicting if names are the same but versions are incompatible. - line 222: // TODO: Mention which one, from labels_to_deps. - line 245: // TODO: Support custom registries for additional packages rust/private/rust_analyzer.bzl (3 lines): - line 110: # TODO: Should this be an error? - line 151: # TODO: Some folks may want to override this for vendored dependencies. - line 172: # TODO: The only imagined use case is an env var holding a filename in the workspace passed to a proto/proto.bzl (2 lines): - line 323: # TODO: Remove once (bazelbuild/bazel#11584) is closed and the rules use - line 401: # TODO: Remove once (bazelbuild/bazel#11584) is closed and the rules use util/label/label.rs (2 lines): - line 9: /// TODO: validate . and .. in target name - line 10: /// TODO: validate used characters in target name crate_universe/src/main.rs (1 line): - line 60: // TODO: Add lockfile versioning and check that here crate_universe/src/config.rs (1 line): - line 121: // TODO: maybe remove the "+buildmetadata" suffix to consolidate e.g. "1.2.3+foo" and "1.2.3". crate_universe/private/util.bzl (1 line): - line 100: # TODO: The resolver triple should be the same as the host but for the time being, tools/runfiles/runfiles.rs (1 line): - line 140: // TODO: 1.28 adds Path::ancestors() which is a little simpler. crate_universe/Cargo.toml (1 line): - line 18: # TODO: Use upstream once following changes are merged: rust/toolchain.bzl (1 line): - line 159: # TODO: Provide a setting to choose between panic_abort and panic_unwind. rust/platform/triple_mappings.bzl (1 line): - line 154: # TODO: This ignores musl. Longer term what does Bazel think about musl? crate_universe/src/renderer.rs (1 line): - line 745: // TODO: Have a single function with kwargs to enable each kind of dep, rather than multiple functions. cargo/cargo_build_script_runner/bin.rs (1 line): - line 192: // TODO: we should consider an alternative to positional arguments. rust/private/utils.bzl (1 line): - line 50: # TODO: Replace with bazel-skylib's `path.dirname`. This requires addressing some