chipsalliance / homebrew-verible

Apache License 2.0
16 stars 8 forks source link

Build/install fails on Ventura 13.3.1 #22

Closed V0XNIHILI closed 1 year ago

V0XNIHILI commented 1 year ago

Expected Behavior

To install verible correctly.

Actual Behavior

brew install verible

==> Fetching dependencies for chipsalliance/verible/verible: openjdk@11
==> Fetching openjdk@11
==> Downloading https://ghcr.io/v2/homebrew/core/openjdk/11/manifests/11.0.19
############################################################################################################### 100.0%
==> Downloading https://ghcr.io/v2/homebrew/core/openjdk/11/blobs/sha256:24d306338e44c638d980a9d028514025a5ccae7970bc4
==> Downloading from https://pkg-containers.githubusercontent.com/ghcr1/blobs/sha256:24d306338e44c638d980a9d028514025a
############################################################################################################### 100.0%
==> Fetching chipsalliance/verible/verible
==> Cloning https://github.com/chipsalliance/verible.git
Updating /Users/douwe/Library/Caches/Homebrew/verible--git
From https://github.com/chipsalliance/verible
 * [new tag]           v0.0-3181-g5a92de1f -> v0.0-3181-g5a92de1f
==> Checking out tag v0.0-3181-g5a92de1f
Previous HEAD position was aac4fadc Merge pull request #1815 from hzeller/20230315-add-macos-cf
HEAD is now at 5a92de1f Merge pull request #1867 from antmicro/static-build-bazel
HEAD is now at 5a92de1f Merge pull request #1867 from antmicro/static-build-bazel
==> Installing verible from chipsalliance/verible
==> Installing dependencies for chipsalliance/verible/verible: openjdk@11
==> Installing chipsalliance/verible/verible dependency: openjdk@11
==> Pouring openjdk@11--11.0.19.ventura.bottle.tar.gz
🍺  /usr/local/Cellar/openjdk@11/11.0.19: 673 files, 298.2MB
==> Installing chipsalliance/verible/verible
==> bazel build --jobs=12 --compilation_mode=opt --copt=-march=native --linkopt=-Wl,-rpath,@loader_path/../lib --verbo
Last 15 lines from /Users/douwe/Library/Logs/Homebrew/verible/01.bazel:
inline void FieldMask::add_paths(std::string&& value) {
                       ^
/usr/local/include/google/protobuf/field_mask.pb.h:286:24: note: candidate function not viable: no known conversion from 'absl::string_view' to 'const char *' for 1st argument
inline void FieldMask::add_paths(const char* value) {
                       ^
/usr/local/include/google/protobuf/field_mask.pb.h:241:32: note: candidate function not viable: requires 0 arguments, but 1 was provided
inline std::string* FieldMask::add_paths() {
                               ^
/usr/local/include/google/protobuf/field_mask.pb.h:291:24: note: candidate function not viable: requires 2 arguments, but 1 was provided
inline void FieldMask::add_paths(const char* value, size_t size) {
                       ^
10 errors generated.
INFO: Elapsed time: 479,389s, Critical Path: 39,63s
INFO: 3779 processes: 1869 internal, 1910 processwrapper-sandbox.
FAILED: Build did NOT complete successfully

If reporting this issue please do so at (not Homebrew/brew or Homebrew/homebrew-core):
  https://github.com/chipsalliance/homebrew-verible/issues

Steps to Reproduce the Problem

  1. brew tap chipsalliance/verible
  2. brew install verible

Specifications

hzeller commented 1 year ago

In #23 we worked towards compiling it on 13.4, so I suspect this can be closed as well.

V0XNIHILI commented 1 year ago

Hey! Thanks for getting back to this! It's indeed fixed now, thanks a lot!!