search

bazelbuild / bazelisk

A user-friendly launcher for Bazel.
Apache License 2.0
2k stars 305 forks source link

Support bazel-watcher (iBazel) #77

Open sandervalstar opened 5 years ago

sandervalstar commented 5 years ago

Are there any plans to support using bazel-watcher through bazelisk?

Adding something like a "--watch" option to bazelisk would be awesome in my opinion.

I'd be happy to try and contribute it myself if given a few pointers on what the solution should look like :)

philwo commented 4 years ago

Hey,

sorry for not following up on this earlier. I just tried to use bazel-watcher with Bazelisk (installed on my PATH as ~/bin/bazel) and it seems to work fine.

What exactly is the problem and what needs to be done here? Is this issue about some incompatibility between bazel-watcher and Bazelisk, or is it a feature request to better integrate bazel-watcher functionality with Bazelisk? :)

gibfahn commented 4 years ago

It might be useful to have bazel-watcher read an env var like IBAZEL_BINARY so you could say IBAZEL_BINARY=bazelisk for those who don't have bazelisk installed as bazel on their machine. Don't think that would require any changes on the bazelisk front though.

zoidyzoidzoid commented 4 years ago

Related to https://github.com/bazelbuild/bazel-watcher/issues/339

lewish commented 4 years ago

There is maybe alternative option here - rather than adding a --watch flag to bazelisk itself, we could have bazelisk also manage ibazel installation. E.g. having a .ibazelversion file, have bazelisk also install a binary called ibazelisk that just downloads and runs the correct version if ibazel, in the same way it runs bazel.

For bonus points, we could also actually install the a symlink for ibazel in the same way it now installs a symlink for bazel, so with npm i -g @bazel/bazelisk you have effectively installed ibazel and bazel with one command.

ghost commented 10 months ago

bump