From 540699f482a1d37ff8481bde8cc81f1fbff3fb3b Mon Sep 17 00:00:00 2001 From: Wangchong Zhou Date: Wed, 11 Sep 2024 11:15:58 +0800 Subject: [PATCH] chore(build): force brotli to statically link --- build/openresty/BUILD.openresty.bazel | 22 ++++++++++++++++------ 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/build/openresty/BUILD.openresty.bazel b/build/openresty/BUILD.openresty.bazel index 4d35d709dd6a..8d09004b6a52 100644 --- a/build/openresty/BUILD.openresty.bazel +++ b/build/openresty/BUILD.openresty.bazel @@ -1,6 +1,7 @@ load("@rules_foreign_cc//foreign_cc:defs.bzl", "configure_make", "make") load("@bazel_skylib//lib:selects.bzl", "selects") load("@kong//build/openresty/wasmx:rules.bzl", "wasm_runtime", "wasmx_configure_options", "wasmx_env") +load("@kong//build:build_system.bzl", "kong_cc_static_library") load("@openresty_binding//:variables.bzl", "LUAJIT_VERSION") load("@kong_bindings//:variables.bzl", "KONG_VAR") load("@kong//build/openresty/wasmx/filters:variables.bzl", "WASM_FILTERS_TARGETS") @@ -233,9 +234,6 @@ CONFIGURE_OPTIONS = [ }) + select({ "@kong//:brotli_flag": [ "--add-module=$$EXT_BUILD_ROOT$$/external/ngx_brotli", - # force static link; the order or following two entries matter - "--with-ld-opt=\"-l:libbrotlienc.a\"", - "--with-ld-opt=\"-l:libbrotlicommon.a\"", ], "//conditions:default": [], }) + wasmx_configure_options @@ -260,6 +258,18 @@ wasm_runtime( visibility = ["//visibility:public"], ) +kong_cc_static_library( + name = "brotlienc", + src = "@brotli//:brotlienc", + visibility = ["//visibility:public"], +) + +kong_cc_static_library( + name = "brotlicommon", + src = "@brotli//:brotlicommon", + visibility = ["//visibility:public"], +) + configure_make( name = "openresty", configure_command = "configure", @@ -309,7 +319,7 @@ configure_make( "//conditions:default": "ln -srf openresty/nginx/sbin/nginx openresty/bin/openresty", }), targets = [ - "-j" + KONG_VAR["NPROC"], + "-j " + KONG_VAR["NPROC"], "install -j" + KONG_VAR["NPROC"], ], visibility = ["//visibility:public"], @@ -330,8 +340,8 @@ configure_make( "//conditions:default": [], }) + select({ "@kong//:brotli_flag": [ - "@brotli//:brotlicommon", - "@brotli//:brotlienc", + ":brotlicommon", + ":brotlienc", ], "//conditions:default": [], }),