Skip to content

Commit

Permalink
fix(build): set $HOME for luarocks build in macos
Browse files Browse the repository at this point in the history
latest `brew` requires $HOME to be set
  • Loading branch information
locao committed Aug 11, 2023
1 parent 7821654 commit 53bf6f5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion build/luarocks/BUILD.luarocks.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ OPENSSL_DIR=$$WORKSPACE_PATH/$$(echo '$(locations @openssl)' | awk '{print $$1}'

# we use system libyaml on macos
if [[ "$$OSTYPE" == "darwin"* ]]; then
YAML_DIR=$$(brew --prefix)/opt/libyaml
YAML_DIR=$$(HOME=~$$(whoami) brew --prefix)/opt/libyaml
elif [[ -d $$WORKSPACE_PATH/$(BINDIR)/external/cross_deps_libyaml/libyaml ]]; then
# TODO: is there a good way to use locations but doesn't break non-cross builds?
YAML_DIR=$$WORKSPACE_PATH/$(BINDIR)/external/cross_deps_libyaml/libyaml
Expand Down

1 comment on commit 53bf6f5

@khcp-gha-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bazel Build

Docker image available kong/kong:53bf6f5739b5b00547b3e7036bd6d79e991be8ed
Artifacts available https://github.com/Kong/kong/actions/runs/5828648924

Please sign in to comment.