build: refactor Bazel configuration towards multiple runtime tests

.bazelrc

@@ -0,0 +1,13 @@
+build --enable_platform_specific_config
+
+# COPTS
+build:linux --cxxopt=-std=c++17
+build:macos --cxxopt=-std=c++17
+# TODO(mathetake): Windows build is not verified yet.
+# build:windows --cxxopt="/std:c++17"
+
+# LINKOPTS
+# See https://bytecodealliance.github.io/wasmtime/c-api/
+build:linux --action_env=BAZEL_LINKOPTS=-lm:-lpthread:-ldl
+# TODO(mathetake): Windows build is not verified yet.
+# build:windows --action_env=BAZEL_LINKOPTS=ws2_32.lib:advapi32.lib:userenv.lib:ntdll.lib:shell32.lib:ole32.lib

.github/workflows/cpp.yml

@@ -56,17 +56,20 @@ jobs:
 
   build:
     runs-on: ubuntu-latest
-
+    strategy:
+      matrix:
+        # TODO(mathetake): Add other runtimes.
+        wasm_runtime: [ "wasmtime" ]
     steps:
     - uses: actions/checkout@v2
 
     - name: Mount bazel cache
       uses: actions/cache@v1
       with:
         path: "/home/runner/.cache/bazel"
-        key: bazel
+        key: bazel-${{ matrix.wasm_runtime }}
 
     - name: Test
       run: |
-        bazel test //...
+        bazel test --define wasm_runtime=${{ matrix.wasm_runtime }} //...
 

BUILD

@@ -1,5 +1,10 @@
 load("@rules_cc//cc:defs.bzl", "cc_library")
-load("@proxy_wasm_cpp_host//bazel:variables.bzl", "COPTS")
+load(
+    "@proxy_wasm_cpp_host//bazel:select.bzl",
+    "proxy_wasm_select_wasm_runtime_v8",
+    "proxy_wasm_select_wasm_runtime_wasmtime",
+    "proxy_wasm_select_wasm_runtime_wavm",
+)
 
 licenses(["notice"])  # Apache 2
 
@@ -13,27 +18,72 @@ cc_library(
     ],
 )
 
-# TODO(mathetkae): once other runtimes(WAVM,V8) can be linked in this repos,
-#   use -define=wasm=v8|wavm|wasmtime and switch
 cc_library(
-    name = "lib",
-    srcs = glob(
-        [
-            "src/**/*.cc",
-            "src/**/*.h",
-        ],
-        exclude = [
-            "src/**/wavm*",
-            "src/**/v8*",
-        ],
-    ),
-    hdrs = glob(["src/**/*.h"]),
-    copts = COPTS,
+    name = "common_lib",
+    srcs = glob([
+        "src/*.h",
+        "src/*.cc",
+        "src/common/*.h",
+        "src/common/*.cc",
+        "src/third_party/*.h",
+        "src/third_party/*.cc",
+        "src/null/*.cc",
+    ]),
     deps = [
         ":include",
         "@boringssl//:crypto",
         "@com_google_protobuf//:protobuf_lite",
         "@proxy_wasm_cpp_sdk//:api_lib",
+    ],
+)
+
+cc_library(
+    name = "v8_lib",
+    srcs = glob([
+        # TODO(@mathetake): Add V8 lib.
+        # "src/v8/*.h",
+        # "src/v8/*.cc",
+    ]),
+    deps = [
+        ":common_lib",
+        # TODO(@mathetake): Add V8 lib.
+    ],
+)
+
+cc_library(
+    name = "wasmtime_lib",
+    srcs = glob([
+        "src/wasmtime/*.h",
+        "src/wasmtime/*.cc",
+    ]),
+    deps = [
+        ":common_lib",
         "@wasm_c_api//:wasmtime_lib",
     ],
 )
+
+cc_library(
+    name = "wavm_lib",
+    srcs = glob([
+        # TODO(@mathetake): Add WAVM lib.
+        # "src/wavm/*.h",
+        # "src/wavm/*.cc",
+    ]),
+    deps = [
+        ":common_lib",
+        # TODO(@mathetake): Add WAVM lib.
+    ],
+)
+
+cc_library(
+    name = "lib",
+    deps = [
+        ":common_lib",
+    ] + proxy_wasm_select_wasm_runtime_wasmtime(
+        ["wasmtime_lib"],
+    ) + proxy_wasm_select_wasm_runtime_v8(
+        ["v8_lib"],
+    ) + proxy_wasm_select_wasm_runtime_wavm(
+        ["wavm_lib"],
+    ),
+)

bazel/BUILD

@@ -0,0 +1,14 @@
+config_setting(
+    name = "wasm_runtime_wavm",
+    values = {"define": "wasm_runtime=wavm"},
+)
+
+config_setting(
+    name = "wasm_runtime_v8",
+    values = {"define": "wasm_runtime=v8"},
+)
+
+config_setting(
+    name = "wasm_runtime_wasmtime",
+    values = {"define": "wasm_runtime=wasmtime"},
+)

bazel/select.bzl

@@ -0,0 +1,31 @@
+# Copyright 2021 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+def proxy_wasm_select_wasm_runtime_v8(xs):
+    return select({
+        "@proxy_wasm_cpp_host//bazel:wasm_runtime_v8": xs,
+        "//conditions:default": [],
+    })
+
+def proxy_wasm_select_wasm_runtime_wavm(xs):
+    return select({
+        "@proxy_wasm_cpp_host//bazel:wasm_runtime_wavm": xs,
+        "//conditions:default": [],
+    })
+
+def proxy_wasm_select_wasm_runtime_wasmtime(xs):
+    return select({
+        "@proxy_wasm_cpp_host//bazel:wasm_runtime_wasmtime": xs,
+        "//conditions:default": [],
+    })

test/BUILD

@@ -1,12 +1,10 @@
 load("@rules_cc//cc:defs.bzl", "cc_library", "cc_test")
-load("@proxy_wasm_cpp_host//bazel:variables.bzl", "COPTS", "LINKOPTS")
 
 package(default_visibility = ["//visibility:public"])
 
 cc_test(
     name = "null_vm_test",
     srcs = ["null_vm_test.cc"],
-    copts = COPTS,
     deps = [
         "//:lib",
         "@com_google_googletest//:gtest",
@@ -17,13 +15,11 @@ cc_test(
 cc_test(
     name = "runtime_test",
     srcs = ["runtime_test.cc"],
-    copts = COPTS,
     data = [
         "//test/test_data:abi_export.wasm",
         "//test/test_data:callback.wasm",
         "//test/test_data:trap.wasm",
     ],
-    linkopts = LINKOPTS,
     deps = [
         ":utility_lib",
         "//:lib",
@@ -35,11 +31,9 @@ cc_test(
 cc_test(
     name = "exports_test",
     srcs = ["exports_test.cc"],
-    copts = COPTS,
     data = [
         "//test/test_data:env.wasm",
     ],
-    linkopts = LINKOPTS,
     deps = [
         ":utility_lib",
         "//:lib",
@@ -51,7 +45,6 @@ cc_test(
 cc_test(
     name = "context_test",
     srcs = ["context_test.cc"],
-    copts = COPTS,
     deps = [
         "//:lib",
         "@com_google_googletest//:gtest",
@@ -62,7 +55,6 @@ cc_test(
 cc_test(
     name = "shared_data",
     srcs = ["shared_data_test.cc"],
-    copts = COPTS,
     deps = [
         "//:lib",
         "@com_google_googletest//:gtest",
@@ -73,7 +65,6 @@ cc_test(
 cc_test(
     name = "shared_queue",
     srcs = ["shared_queue_test.cc"],
-    copts = COPTS,
     deps = [
         "//:lib",
         "@com_google_googletest//:gtest",
@@ -84,7 +75,6 @@ cc_test(
 cc_test(
     name = "vm_id_handle",
     srcs = ["vm_id_handle_test.cc"],
-    copts = COPTS,
     deps = [
         "//:lib",
         "@com_google_googletest//:gtest",
@@ -99,7 +89,6 @@ cc_library(
         "utility.h",
     ],
     hdrs = ["utility.h"],
-    copts = COPTS,
     deps = [
         "//:lib",
         "@com_google_googletest//:gtest",

test/common/BUILD

@@ -1,10 +1,8 @@
 load("@rules_cc//cc:defs.bzl", "cc_test")
-load("@proxy_wasm_cpp_host//bazel:variables.bzl", "COPTS")
 
 cc_test(
     name = "bytecode_util_test",
     srcs = ["bytecode_util_test.cc"],
-    copts = COPTS,
     data = [
         "//test/test_data:abi_export.wasm",
     ],