Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Update to a more recent rules rust and update to use defs.bzl #1955

Merged
merged 1 commit into from
Nov 9, 2021

Conversation

dfreese
Copy link
Contributor

@dfreese dfreese commented Nov 8, 2021

Rules rust has made some changes upstream that remove the rust.bzl file.
It had previously been left in there for compatibility purposes. This
has been ongoing since February 16th. See this issue for additional
context: bazelbuild/rules_rust#591. Replaces
PRs #1950 and #1951.

Fixes #1949

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

Rules rust has made some changes upstream that remove the rust.bzl file.
It had previously been left in there for compatibility purposes.  This
has been ongoing since February 16th.  See this issue for additional
context: bazelbuild/rules_rust#591.
@dfreese
Copy link
Contributor Author

dfreese commented Nov 8, 2021

@gravypod this should be good-to-go.

Copy link
Collaborator

@gravypod gravypod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gravypod gravypod merged commit 40cdf2d into bazelbuild:master Nov 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rules Docker no longer works with the latest commit of rules_rust
2 participants