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

Extract a method for locating Bazel #635

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shs96c
Copy link

@shs96c shs96c commented Nov 20, 2024

bazelisk can be thought of as two separate pieces. The first of these is a mechanism for locating a particular bazel version, possibly downloading it from the Web, or using a local install. The second piece is the additional functionality for flag flipping and finding out where problems have occurred within commits to the project.

This PR extracts the location of the bazel binary into its own method which can be used by other tooling that also needs to locate the binary.

`bazelisk` can be thought of as two separate pieces. The first of these is a
mechanism for locating a particular `bazel` version, possibly downloading it
from the Web, or using a local install. The second piece is the additional
functionality for flag flipping and finding out where problems have occurred
within commits to the project.

This PR extracts the location of the `bazel` binary into its own method which
can be used by other tooling that also needs to locate the binary.
@shs96c
Copy link
Author

shs96c commented Nov 21, 2024

Running the failing //:go_bazelisk_test test on HEAD of master locally also times out for me. I'm not sure the test failure is related to this PR.

@hauserx
Copy link

hauserx commented Nov 21, 2024

Running the failing //:go_bazelisk_test test on HEAD of master locally also times out for me. I'm not sure the test failure is related to this PR.

You maybe just unlucky, this test is running close to 5 min default limit. Also noticed this issue locally and created #637 to fix it.

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