build: exclude liburing from non-Linux builds (envoyproxy#29635)

This fixes some uncommon bazel invocations such as `bazel build //source/...` and `tools/gen_compilation_database.py` on non-Linux. Signed-off-by: Greg Greenway <[email protected]>
kyessenov · Sep 14, 2023 · ef420f5 · ef420f5
1 parent 2add6ca
commit ef420f5
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 6 deletions.
diff --git a/bazel/foreign_cc/BUILD b/bazel/foreign_cc/BUILD
@@ -1,5 +1,5 @@
 load("@rules_foreign_cc//foreign_cc:configure.bzl", "configure_make")
-load("//bazel:envoy_build_system.bzl", "envoy_cmake", "envoy_package")
+load("//bazel:envoy_build_system.bzl", "envoy_cc_library", "envoy_cmake", "envoy_package")
 
 licenses(["notice"])  # Apache 2
 
@@ -17,6 +17,15 @@ configure_make(
     ],
 )
 
+envoy_cc_library(
+    name = "liburing_linux",
+    srcs = [],
+    deps = select({
+        "//bazel:linux": [":liburing"],
+        "//conditions:default": [],
+    }),
+)
+
 # autotools packages are unusable on Windows as-is
 # TODO: Consider our own gperftools.BUILD file as we do with many other packages
 configure_make(

diff --git a/bazel/repositories.bzl b/bazel/repositories.bzl
@@ -388,7 +388,7 @@ def _com_github_axboe_liburing():
     )
     native.bind(
         name = "uring",
-        actual = "@envoy//bazel/foreign_cc:liburing",
+        actual = "@envoy//bazel/foreign_cc:liburing_linux",
     )
 
 def _com_github_bazel_buildtools():

diff --git a/source/common/io/BUILD b/source/common/io/BUILD
@@ -10,9 +10,10 @@ envoy_package()
 
 envoy_cc_library(
     name = "io_uring_impl_lib",
-    srcs = [
-        "io_uring_impl.cc",
-    ],
+    srcs = select({
+        "//bazel:linux": ["io_uring_impl.cc"],
+        "//conditions:default": [],
+    }),
     hdrs = [
         "io_uring_impl.h",
     ],
@@ -26,7 +27,10 @@ envoy_cc_library(
 
 envoy_cc_library(
     name = "io_uring_worker_lib",
-    srcs = ["io_uring_worker_impl.cc"],
+    srcs = select({
+        "//bazel:linux": ["io_uring_worker_impl.cc"],
+        "//conditions:default": [],
+    }),
     hdrs = ["io_uring_worker_impl.h"],
     deps = [
         ":io_uring_impl_lib",