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

refactor: download bazelisk to /usr/local/bin/bazel #1507

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

Conversation

Aghassi
Copy link

@Aghassi Aghassi commented Jun 18, 2022

The original implementation of this links bazelisk to /user/local/bin/bazelisk, but that means a user would have to type bazelisk to have it install bazel. The more natural way for a user to interact in a bazel project is just to run bazel. If bazelisk is linked to the bazel path it will automatically fetch bazel for the end user and not have to worry about different commands.

Here is an ergonmic example of what I expect as an end user when I start a space using this file

@Aghassi ➜ /workspaces/bazel-lib (vscodepsaces/dockerfile ✗) $ which bazel
/usr/local/bin/bazel
@Aghassi ➜ /workspaces/bazel-lib (vscodepsaces/dockerfile ✗) $ bazel
2022/06/18 19:05:05 Downloading https://releases.bazel.build/5.0.0/rolling/5.0.0-pre.20211011.2/bazel-5.0.0-pre.20211011.2-linux-x86_64...
Extracting Bazel installation...

The original implementation of this links `bazelisk` to `/user/local/bin/bazelisk`, but that means a user would have to type `bazelisk` to have it install bazel. The more natural way for a user to interact in a `bazel` project is just to run `bazel`. If `bazelisk` is linked to the `bazel` path it will automatically fetch `bazel` for the end user and not have to worry about different commands.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant