Make it easier to use Rust rules with different versions of Rust #14
Assignees
Labels
Comments
davidzchen
referenced
this issue
in kythe/kythe
Jun 29, 2016
Summary: Prepping for the implementation of the rust indexer, this commit sets up bazel builds for rust. Currently it only builds a hello world program but the project is split into two different components a library and an executable. Test Plan: Run bazel build //kythe/rust/indexer:release. If rust is present it will build, and //kythe/rust/indexer:test will run the tests. If rust is not present both will fail and display an error message. Reviewers: zarko, schroederc Reviewed By: schroederc Projects: #rust Differential Revision: https://phabricator-dot-kythe-repo.appspot.com/D931
|
+cc @katre |
|
I am working on this as part of my ongoing work on Bazel Platform support, so I will take this issue. |
|
Sounds awesome. Thanks! |
katre
added a commit
that referenced
this issue
Sep 27, 2017
This allows rust targets to only depend on (and thus download) the toolchains that are used, dynamically chosen based on the current target platform. Fixes #14.
acmcarther
added a commit
to acmcarther/rules_rust
that referenced
this issue
Jan 20, 2018
* Make rule tests compatible with the new string representations in Skylark (bazelbuild#51) * Rename set to depset (bazelbuild#50) * Update rules_rust to use Bazel's toolchains feature. (bazelbuild#52) This allows rust targets to only depend on (and thus download) the toolchains that are used, dynamically chosen based on the current target platform. Fixes bazelbuild#14. * Sketch of dylib support * Update compiler toolchain to 1.20.0. (bazelbuild#56) * Fix transitive dylib support. * Fix 'data' field not being made available to rust_test rule. * Topological sort for dylib link flags * Tidy rpath fn * Fix linking transitive static libs, and dylib ordering. * Actually export rust_library.dynamic_libs * Make sure TMPDIR exists for rustc. * Properly include dynamic_libs in runfiles. * fix syntax errors * more * Add support for depending on shared libraries on linux. * Merge raze patches into dylibs branch. * Fixup merge, including remove broken ctx.file.out_dir_tar from doc_* rules. * look for native deps when only transitive * deduplicate rpaths * hackily fix .so symlink..
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, the version of Rust used by the rules are not easily configurable. We should make it easier to use the Rust rules with different Rust versions, such as via a
rust_toolchainrule (similar tocc_toolchain) or look at the approach taken by tools such as multirust.It would also be good to make it easy to switch between having Bazel download a set of Rust binaries and using the ones installed on the local system.
The text was updated successfully, but these errors were encountered: