Cannot build with bazel #27

derekperkins · 2023-08-14T03:08:59Z

Description

Build with bazel after adding protovalidate-go. Go builds succeed, but bazel builds fail on cel-go references. No proto compilation happens inside bazel, generated files are committed.

Screenshots/Logs

'@com_github_google_cel_go//cel:cel': target 'cel' not declared in package 'cel' defined by /external/com_github_google_cel_go/cel/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/celext:celext'
'@com_github_google_cel_go//common/overloads:overloads': target 'overloads' not declared in package 'common/overloads' defined by /external/com_github_google_cel_go/common/overloads/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/celext:celext'
'@com_github_google_cel_go//common/types:types': target 'types' not declared in package 'common/types' defined by /external/com_github_google_cel_go/common/types/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/celext:celext'
'@com_github_google_cel_go//common/types/ref:ref': target 'ref' not declared in package 'common/types/ref' defined by /external/com_github_google_cel_go/common/types/ref/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/celext:celext'
'@com_github_google_cel_go//common/types/traits:traits': target 'traits' not declared in package 'common/types/traits' defined by /external/com_github_google_cel_go/common/types/traits/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/celext:celext'
'@com_github_google_cel_go//ext:ext': target 'ext' not declared in package 'ext' defined by /external/com_github_google_cel_go/ext/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/celext:celext'
'@com_github_google_cel_go//cel:cel': target 'cel' not declared in package 'cel' defined by /external/com_github_google_cel_go/cel/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/expression:expression'
'@com_github_google_cel_go//interpreter:interpreter': target 'interpreter' not declared in package 'interpreter' defined by /external/com_github_google_cel_go/interpreter/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/expression:expression'
'@com_github_google_cel_go//cel:cel': target 'cel' not declared in package 'cel' defined by /external/com_github_google_cel_go/cel/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/constraints:constraints'
'@com_github_google_cel_go//cel:cel': target 'cel' not declared in package 'cel' defined by /external/com_github_google_cel_go/cel/BUILD.bazel and referenced by '@com_github_bufbuild_protovalidate_go//internal/evaluator:evaluator'

Environment

protovalidate-go v0.2.1
bazel v6.3.2
go v1.21.0
rules_go v0.41.0
gazelle v0.32.0
github.com/google/cel-go v0.17.1

Additional Context

Trouble building with bazel google/cel-go#801

The text was updated successfully, but these errors were encountered:

Alfus · 2023-08-14T18:49:00Z

How are you using bazel to build protovalidate-go? It seems like there is an issue in whatever is creating those build files. It looks like it is referencing non-existing targets in cel-go. For example, looking at cel-go BUILD file https://github.com/google/cel-go/blob/master/cel/BUILD.bazel#L8, it seems like @com_github_google_cel_go//cel:cel should actually be @com_github_google_cel_go//cel:go_default_library

derekperkins · 2023-08-14T19:00:54Z

I'm just using https://github.com/bazelbuild/bazel-gazelle/ to generate the build files from our go.mod, like for the rest of our repo. This is how it is referenced in our deps.bzl file

    go_repository(
        name = "com_github_google_cel_go",
        build_file_proto_mode = "disable_global",
        importpath = "github.com/google/cel-go",
        sum = "h1:s2151PDGy/eqpCI80/8dl4VL3xTkqI/YubXLXCFw0mw=",
        version = "v0.17.1",
    )

Here are the references to protovalidate

    go_repository(
        name = "build_buf_gen_go_bufbuild_protovalidate_protocolbuffers_go",
        build_file_proto_mode = "disable_global",
        importpath = "buf.build/gen/go/bufbuild/protovalidate/protocolbuffers/go",
        sum = "h1:mnhf3O5uBs95ngTaQbGZfAnoZC0lM6yWkpdgjtqPbNE=",
        version = "v1.31.0-20230721003620-2341cbb21958.1",
    )

    go_repository(
        name = "com_github_bufbuild_protovalidate_go",
        build_file_proto_mode = "disable_global",
        importpath = "github.com/bufbuild/protovalidate-go",
        sum = "h1:pJr07sYhliyfj/STAM7hU4J3FKpVeLVKvOBmOTN8j+s=",
        version = "v0.2.1",
    )

derekperkins · 2023-08-15T20:48:38Z

It feels likely that this is at least partially interacting with gazelle, so here are my settings in case that's useful

# Exclude directories from gazelle
# gazelle:exclude build
# gazelle:exclude db
# gazelle:exclude hack

# gazelle:prefix go.nozzle.io
# gazelle:go_naming_convention go_default_library
# gazelle:go_generate_proto false
# gazelle:proto disable_global

# default gazelle run to update all BUILD files
gazelle(name = "gazelle")

# syncs go.mod to deps.bzl
gazelle(
    name = "gazelle-update-repos",
    args = [
        "-from_file=go.mod",
        "-to_macro=deps.bzl%go_dependencies",
        "-prune",
        "-build_file_proto_mode=disable_global",
    ],
    command = "update-repos",
)

derekperkins · 2023-08-15T21:18:22Z

I was able to track down the issue and get it fixed with a workaround. The actual issue is that gazelle isn't quite smart enough to handle repos like cel-go that already have bazel files. For anyone who finds this issue, the workaround is to manually add build_naming_convention = "go_default_library", to to the go_repository.

    go_repository(
        name = "com_github_google_cel_go",
        build_file_proto_mode = "disable_global",
        build_naming_convention = "go_default_library",
        importpath = "github.com/google/cel-go",
        sum = "h1:s2151PDGy/eqpCI80/8dl4VL3xTkqI/YubXLXCFw0mw=",
        version = "v0.17.1",
    )

If this underlying issue with gazelle is resolved, this manual workaround shouldn't be needed in the future.

update-repos: set build_naming_convention for modules that already have build files bazel-contrib/bazel-gazelle#890

debkanchan · 2023-12-05T06:06:21Z

I'm using the upcoming bzlmod support in rules_buf and I've encountered this error.

ERROR: /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go/celext/BUILD.bazel:3:11: no such target '@gazelle~0.34.0~go_deps~com_github_google_cel_go//cel:cel': target 'cel' not declared in package 'cel' defined by /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_google_cel_go/cel/BUILD.bazel (Tip: use `query "@@gazelle~0.34.0~go_deps~com_github_google_cel_go//cel:*"` to see all the targets in that package) and referenced by '@gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go//celext:celext'
ERROR: /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go/celext/BUILD.bazel:3:11: no such target '@gazelle~0.34.0~go_deps~com_github_google_cel_go//common/overloads:overloads': target 'overloads' not declared in package 'common/overloads' defined by /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_google_cel_go/common/overloads/BUILD.bazel (did you mean 'overloads.go'? Tip: use `query "@@gazelle~0.34.0~go_deps~com_github_google_cel_go//common/overloads:*"` to see all the targets in that package) and referenced by '@gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go//celext:celext'
ERROR: /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go/celext/BUILD.bazel:3:11: no such target '@gazelle~0.34.0~go_deps~com_github_google_cel_go//common/types:types': target 'types' not declared in package 'common/types' defined by /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_google_cel_go/common/types/BUILD.bazel (Tip: use `query "@@gazelle~0.34.0~go_deps~com_github_google_cel_go//common/types:*"` to see all the targets in that package) and referenced by '@gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go//celext:celext'
ERROR: /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go/celext/BUILD.bazel:3:11: no such target '@gazelle~0.34.0~go_deps~com_github_google_cel_go//common/types/ref:ref': target 'ref' not declared in package 'common/types/ref' defined by /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_google_cel_go/common/types/ref/BUILD.bazel (Tip: use `query "@@gazelle~0.34.0~go_deps~com_github_google_cel_go//common/types/ref:*"` to see all the targets in that package) and referenced by '@gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go//celext:celext'
ERROR: /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go/celext/BUILD.bazel:3:11: no such target '@gazelle~0.34.0~go_deps~com_github_google_cel_go//common/types/traits:traits': target 'traits' not declared in package 'common/types/traits' defined by /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_google_cel_go/common/types/traits/BUILD.bazel (Tip: use `query "@@gazelle~0.34.0~go_deps~com_github_google_cel_go//common/types/traits:*"` to see all the targets in that package) and referenced by '@gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go//celext:celext'
ERROR: /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go/celext/BUILD.bazel:3:11: no such target '@gazelle~0.34.0~go_deps~com_github_google_cel_go//ext:ext': target 'ext' not declared in package 'ext' defined by /private/var/tmp/_bazel_debkanchan/969f704a68afbe6aa6f76e61ecfa9c99/external/gazelle~0.34.0~go_deps~com_github_google_cel_go/ext/BUILD.bazel (Tip: use `query "@@gazelle~0.34.0~go_deps~com_github_google_cel_go//ext:*"` to see all the targets in that package) and referenced by '@gazelle~0.34.0~go_deps~com_github_bufbuild_protovalidate_go//celext:celext'

MODULE

bazel_dep(name = "rules_go", version = "0.42.0")
bazel_dep(name = "gazelle", version = "0.34.0")
bazel_dep(name = "rules_pkg", version = "0.9.1")
bazel_dep(name = "rules_buf", version = "0.2.0")
bazel_dep(name = "rules_proto", version = "5.3.0-21.7")
bazel_dep(name = "rules_oci", version = "1.4.0")
bazel_dep(name = "container_structure_test", version = "1.16.0")

git_override(
    module_name = "rules_buf",
    commit = "27e9a19c6227cc3923abb1a4e703d9d42d9a1fa9",
    remote = "https://github.com/bufbuild/rules_buf.git",
)

buf = use_extension("@rules_buf//buf:extensions.bzl", "buf")

# Override the default version of buf
buf.toolchains(version = "v1.28.1")

go_deps = use_extension("@gazelle//:extensions.bzl", "go_deps")
go_deps.from_file(go_mod = "//:go.mod")
use_repo(go_deps, "build_buf_gen_go_bufbuild_protovalidate_protocolbuffers_go", "build_buf_gen_go_ride_wallet_connectrpc_go", "build_buf_gen_go_ride_wallet_protocolbuffers_go", "com_connectrpc_connect", "com_github_bufbuild_protovalidate_go", "com_github_golang_jwt_jwt_v5", "com_github_google_wire", "com_github_ilyakaznacheev_cleanenv", "com_github_micahparks_keyfunc_v2", "com_github_mmcloughlin_geohash", "com_github_onsi_ginkgo_v2", "com_github_onsi_gomega", "com_google_cloud_go_firestore", "com_google_firebase_go_v4", "org_golang_google_genproto", "org_golang_google_genproto_googleapis_api", "org_golang_google_grpc", "org_golang_google_protobuf", "org_golang_x_net", "org_uber_go_mock", "org_uber_go_zap")

oci = use_extension("@rules_oci//oci:extensions.bzl", "oci")
oci.pull(
    name = "distroless",
    digest = "sha256:e7e79fb2947f38ce0fab6061733f7e1959c12b843079042fe13f56ca7b9d178c",
    image = "gcr.io/distroless/static",
    platforms = [
        "linux/amd64",
        "linux/arm64/v8",
    ],
)
use_repo(oci, "distroless")

BUILD

load("@gazelle//:def.bzl", "DEFAULT_LANGUAGES", "gazelle", "gazelle_binary")

gazelle_binary(
    name = "gazelle-buf",
    languages = DEFAULT_LANGUAGES + [
        # Loads the Buf extension
        "@rules_buf//gazelle/buf:buf",
        # NOTE: This needs to be loaded after the proto language
    ],
)

# gazelle:prefix github.com/ride-app/driver-service
# gazelle:build_file_name BUILD.bazel
# gazelle:exclude infra
# gazelle:proto disable_global
gazelle(
    name = "gazelle",
    gazelle = ":gazelle-buf",
)

Since bzlmod support means there's no go_deps.bazel file anymore where I could manually change settings I'm stuck as it seems.

debkanchan · 2023-12-05T06:19:22Z

@srikrsna-buf

debkanchan · 2023-12-13T18:06:25Z

@srikrsna-buf until the fix is there any workaround?

srikrsna-buf · 2023-12-20T09:35:00Z

@DebkanchanSamadder Sorry I was out and just getting back and I'll try and replicate and get back to you

rodaine · 2023-12-20T16:40:03Z

@DebkanchanSamadder can you create a new issue specific to this bzlmod concern?

debkanchan · 2023-12-21T16:37:10Z

#88 @rodaine

derekperkins added the Bug Something isn't working label Aug 14, 2023

derekperkins mentioned this issue Aug 15, 2023

Gazelle is not respecting the go_naming_convention directive for external dependencies bazel-contrib/bazel-gazelle#1557

Closed

derekperkins closed this as completed Aug 15, 2023

debkanchan mentioned this issue Dec 1, 2023

Support gazelle in bzlmod bufbuild/rules_buf#46

Merged

debkanchan mentioned this issue Dec 6, 2023

update-repos: set build_naming_convention for modules that already have build files bazel-contrib/bazel-gazelle#890

Open

debkanchan mentioned this issue Dec 21, 2023

[BUG] Cannot build with Bzlmod #88

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Cannot build with bazel #27

Cannot build with bazel #27

derekperkins commented Aug 14, 2023 •

edited

Loading

Alfus commented Aug 14, 2023 •

edited

Loading

derekperkins commented Aug 14, 2023 •

edited

Loading

derekperkins commented Aug 15, 2023 •

edited

Loading

derekperkins commented Aug 15, 2023

debkanchan commented Dec 5, 2023 •

edited

Loading

debkanchan commented Dec 5, 2023

debkanchan commented Dec 13, 2023

srikrsna-buf commented Dec 20, 2023

rodaine commented Dec 20, 2023

debkanchan commented Dec 21, 2023

Cannot build with bazel #27

Cannot build with bazel #27

Comments

derekperkins commented Aug 14, 2023 • edited Loading

Description

Screenshots/Logs

Environment

Additional Context

Alfus commented Aug 14, 2023 • edited Loading

derekperkins commented Aug 14, 2023 • edited Loading

derekperkins commented Aug 15, 2023 • edited Loading

derekperkins commented Aug 15, 2023

debkanchan commented Dec 5, 2023 • edited Loading

debkanchan commented Dec 5, 2023

debkanchan commented Dec 13, 2023

srikrsna-buf commented Dec 20, 2023

rodaine commented Dec 20, 2023

debkanchan commented Dec 21, 2023

derekperkins commented Aug 14, 2023 •

edited

Loading

Alfus commented Aug 14, 2023 •

edited

Loading

derekperkins commented Aug 14, 2023 •

edited

Loading

derekperkins commented Aug 15, 2023 •

edited

Loading

debkanchan commented Dec 5, 2023 •

edited

Loading