Skip to content

Rename set to depset #50

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

Merged
merged 1 commit into from
Aug 7, 2017
Merged

Rename set to depset #50

merged 1 commit into from
Aug 7, 2017

Conversation

vladmos
Copy link
Member

@vladmos vladmos commented Jul 26, 2017

No description provided.

@vladmos vladmos requested a review from katre August 7, 2017 15:34
@katre katre merged commit 9c8b8d9 into bazelbuild:master Aug 7, 2017
@mfarrugi mfarrugi mentioned this pull request Nov 4, 2017
acmcarther added a commit to acmcarther/rules_rust that referenced this pull request 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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants