Skip to content
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

Add windows_msvc back to conditions in bazel_tools. #12668

Closed
wants to merge 1 commit into from

Conversation

comius
Copy link
Contributor

@comius comius commented Dec 9, 2020

This is partial revert of 6d637f4 and a fix to a06d7e1, which went into wrong file.

windows_msvc condition is used downstream by tensorflow via ruy.

The culprit line is in https://github.com/google/ruy/blob/master/ruy/build_defs.bzl#L60 (and #L67,#L77).

This is partial revert of 6d637f4 and a fix to a06d7e1, which went into wrong file.

windows_msvc condition is used downstream by tensorflow via ruy.

The culprit line is in https://github.com/google/ruy/blob/master/ruy/build_defs.bzl#L60 (and #L67,#L77).
@comius comius requested a review from katre December 9, 2020 08:50
@google-cla google-cla bot added the cla: yes label Dec 9, 2020
@bazel-io bazel-io closed this in 88b1cb4 Dec 9, 2020
meisterT pushed a commit that referenced this pull request Jan 13, 2021
This is partial revert of 6d637f4 and a fix to a06d7e1, which went into wrong file.

windows_msvc condition is used downstream by tensorflow via ruy.

The culprit line is in https://github.com/google/ruy/blob/master/ruy/build_defs.bzl#L60 (and #L67,#L77).

Closes #12668.

PiperOrigin-RevId: 346540473
(cherry picked from commit 88b1cb4)
meisterT pushed a commit that referenced this pull request Jan 13, 2021
This is partial revert of 6d637f4 and a fix to a06d7e1, which went into wrong file.

windows_msvc condition is used downstream by tensorflow via ruy.

The culprit line is in https://github.com/google/ruy/blob/master/ruy/build_defs.bzl#L60 (and #L67,#L77).

Closes #12668.

PiperOrigin-RevId: 346540473
ulfjack pushed a commit to EngFlow/bazel that referenced this pull request Mar 5, 2021
This is partial revert of 6d637f4 and a fix to a06d7e1, which went into wrong file.

windows_msvc condition is used downstream by tensorflow via ruy.

The culprit line is in https://github.com/google/ruy/blob/master/ruy/build_defs.bzl#L60 (and #L67,#L77).

Closes bazelbuild#12668.

PiperOrigin-RevId: 346540473
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.

2 participants