Skip to content

bazel: change .bazelversion to be symlink from envoy#1925

Merged
goaway merged 1 commit intomainfrom
ks/bazel-change-.bazelversion-to-be-symlink-from-envoy
Nov 18, 2021
Merged

bazel: change .bazelversion to be symlink from envoy#1925
goaway merged 1 commit intomainfrom
ks/bazel-change-.bazelversion-to-be-symlink-from-envoy

Conversation

@keith
Copy link
Member

@keith keith commented Nov 6, 2021

These should likely always match since envoy's build configs could
depend on it.

Signed-off-by: Keith Smiley keithbsmiley@gmail.com

@keith keith marked this pull request as ready for review November 6, 2021 01:34
These should likely always match since envoy's build configs could
depend on it.

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
@keith keith force-pushed the ks/bazel-change-.bazelversion-to-be-symlink-from-envoy branch from 08fda85 to 87f3c91 Compare November 10, 2021 02:19
@goaway
Copy link
Contributor

goaway commented Nov 18, 2021

Thanks @keith! This sounds good to me. We have historically had a couple of stretches where we needed to build on a different version of bazel from the version on Envoy main, but this seems good to have as a default and it's not like we can't change it if we're forced to down the road.

@goaway goaway merged commit 9aeaaa5 into main Nov 18, 2021
@goaway goaway deleted the ks/bazel-change-.bazelversion-to-be-symlink-from-envoy branch November 18, 2021 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants