From 2ec1d5c48f8ebd8b0e996982f562e4937728b4dc Mon Sep 17 00:00:00 2001 From: Greg Magolan Date: Thu, 17 Nov 2022 04:45:44 -0800 Subject: [PATCH 1/2] chore(cli): make it more clear on how to install aspect on Linux (#811) GitOrigin-RevId: 6309af05572b3a3633dd79665bbfbe53e052d629 --- README.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/README.md b/README.md index eb4a1aa98..5eeae23ef 100644 --- a/README.md +++ b/README.md @@ -26,6 +26,10 @@ This installs the `aspect` command and also links it to `bazel`, just like the [ > We plan to have a standard "core" homebrew formula so this will just be `brew install aspect` in the future. +## Linux + +On Linux, you can download the `aspect` binary on our [Releases](https://github.com/aspect-build/aspect-cli/releases) page and add it to your `PATH` manually. This also works on MacOS and Windows. + ## Bazelisk On any platform, so long as you already have [bazelisk] installed, you can have [bazelisk] From 81a5adbb15d903afff97e55586641028ea64f277 Mon Sep 17 00:00:00 2001 From: Greg Magolan Date: Thu, 17 Nov 2022 04:46:43 -0800 Subject: [PATCH 2/2] fix(cli): allow any execute bits to count as executable (#810) GitOrigin-RevId: 8423ed120c8a53e3729a98feee915632c46f2a7f --- pkg/bazel/bazelisk.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/bazel/bazelisk.go b/pkg/bazel/bazelisk.go index a5e811a41..db1c628c2 100644 --- a/pkg/bazel/bazelisk.go +++ b/pkg/bazel/bazelisk.go @@ -456,7 +456,7 @@ func (bazelisk *Bazelisk) maybeDelegateToWrapper(bazel string) string { } wrapper := filepath.Join(bazelisk.workspaceRoot, wrapperPath) - if stat, err := os.Stat(wrapper); err != nil || stat.IsDir() || stat.Mode().Perm()&0001 == 0 { + if stat, err := os.Stat(wrapper); err != nil || stat.IsDir() || stat.Mode().Perm()&0111 == 0 { return bazel }