Skip to content

Commit

Permalink
Remove linux go build constraint for uffd (#8042)
Browse files Browse the repository at this point in the history
We don't need the go build constraint because the BUILD file contains a
`target_compatible_with` platform. This makes it easier to edit the file
with an IDE, even on non-linux platforms

This was originally added here
#7965, then reverted due
to the separate changes in that PR
  • Loading branch information
maggie-lou authored Dec 11, 2024
1 parent 72acec5 commit bd7711c
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 13 deletions.
19 changes: 8 additions & 11 deletions enterprise/server/remote_execution/uffd/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -9,17 +9,14 @@ go_library(
"@platforms//os:linux",
],
visibility = ["//visibility:public"],
deps = select({
"@io_bazel_rules_go//go/platform:linux": [
"//enterprise/server/remote_execution/copy_on_write",
"//server/metrics",
"//server/util/log",
"//server/util/status",
"@com_github_prometheus_client_golang//prometheus",
"@org_golang_x_sys//unix",
],
"//conditions:default": [],
}),
deps = [
"//enterprise/server/remote_execution/copy_on_write",
"//server/metrics",
"//server/util/log",
"//server/util/status",
"@com_github_prometheus_client_golang//prometheus",
"@org_golang_x_sys//unix",
],
)

package(default_visibility = ["//enterprise:__subpackages__"])
2 changes: 0 additions & 2 deletions enterprise/server/remote_execution/uffd/uffd.go
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
//go:build linux && !android

package uffd

import (
Expand Down

0 comments on commit bd7711c

Please sign in to comment.