Merge pull request #58 from kroma-network/build/check-node-buildable-…

…using-bazel-condition

build: check node buildable using bazel condition

tachyon/node/BUILD.bazel

@@ -1,8 +1,9 @@
+load("//bazel:tachyon.bzl", "if_node_binding")
 load("//bazel:tachyon_node.bzl", "tachyon_node_library")
 
 tachyon_node_library(
     name = "tachyon",
-    srcs = ["tachyon.cc"],
+    srcs = if_node_binding(["tachyon.cc"]),
     deps = [
         "//tachyon/node/math",
     ],

tachyon/node/base/BUILD.bazel

@@ -3,7 +3,7 @@ load("//bazel:tachyon_cc.bzl", "tachyon_cc_library")
 
 tachyon_cc_library(
     name = "node_base",
-    srcs = [
+    srcs = if_node_binding([
         "node_constructors.cc",
         "node_constructors.h",
         "node_cpp_bindable.cc",
@@ -18,7 +18,7 @@ tachyon_cc_library(
         "node_internals.h",
         "node_internals_forwards.h",
         "node_module.cc",
-    ],
+    ]),
     hdrs = [
         "node_cpp_class.h",
         "node_cpp_enum.h",

tachyon/node/base/node_constructors.cc

@@ -1,5 +1,3 @@
-#if defined(TACHYON_NODE_BINDING)
-
 #include "tachyon/node/base/node_constructors.h"
 
 namespace tachyon::node {
@@ -35,5 +33,3 @@ bool NodeConstructors::InstanceOf(Napi::Object object, std::string_view name) {
 }
 
 }  // namespace tachyon::node
-
-#endif  // defined(TACHYON_NODE_BINDING)

tachyon/node/base/node_constructors.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_CONSTRUCTORS_H_
 #define TACHYON_NODE_BASE_NODE_CONSTRUCTORS_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <string>
 #include <string_view>
 
@@ -38,6 +36,4 @@ class TACHYON_EXPORT NodeConstructors {
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_CONSTRUCTORS_H_

tachyon/node/base/node_cpp_bindable.cc

@@ -1,5 +1,3 @@
-#if defined(TACHYON_NODE_BINDING)
-
 #include "tachyon/node/base/node_cpp_bindable.h"
 
 #include "absl/strings/substitute.h"
@@ -27,5 +25,3 @@ NodeCppBindable& NodeCppBindable::operator=(NodeCppBindable&& other) noexcept =
 NodeCppBindable::~NodeCppBindable() = default;
 
 }  // namespace tachyon::node
-
-#endif  // defined(TACHYON_NODE_BINDING)

tachyon/node/base/node_cpp_bindable.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_CPP_BINDABLE_H_
 #define TACHYON_NODE_BASE_NODE_CPP_BINDABLE_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <string>
 #include <string_view>
 
@@ -32,6 +30,4 @@ class TACHYON_EXPORT NodeCppBindable {
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_CPP_BINDABLE_H_

tachyon/node/base/node_cpp_callable.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_CPP_CALLABLE_H_
 #define TACHYON_NODE_BASE_NODE_CPP_CALLABLE_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <tuple>
 
 #include "absl/strings/substitute.h"
@@ -346,6 +344,4 @@ class NodeCppCallable {
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_CPP_CALLABLE_H_

tachyon/node/base/node_cpp_callable_data.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_CPP_CALLABLE_DATA_H_
 #define TACHYON_NODE_BASE_NODE_CPP_CALLABLE_DATA_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <type_traits>
 #include <vector>
 
@@ -63,6 +61,4 @@ struct TACHYON_EXPORT NodeCppStaticPropertyAccessorData
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_CPP_CALLABLE_DATA_H_

tachyon/node/base/node_cpp_class.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_CPP_CLASS_H_
 #define TACHYON_NODE_BASE_NODE_CPP_CLASS_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <string_view>
 
 #include "absl/strings/substitute.h"
@@ -368,6 +366,4 @@ class NodeCppClass : public NodeCppBindable {
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_CPP_CLASS_H_

tachyon/node/base/node_cpp_constructor.cc

@@ -1,5 +1,3 @@
-#if defined(TACHYON_NODE_BINDING)
-
 #include "tachyon/node/base/node_cpp_constructor.h"
 
 #include "tachyon/base/strings/string_util.h"
@@ -75,5 +73,3 @@ bool NodeCppConstructor::Match(const Napi::CallbackInfo& info) const {
 }
 
 }  // namespace tachyon::node
-
-#endif  // defined(TACHYON_NODE_BINDING)

tachyon/node/base/node_cpp_constructor.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_CPP_CONSTRUCTOR_H_
 #define TACHYON_NODE_BASE_NODE_CPP_CONSTRUCTOR_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <string>
 #include <vector>
 
@@ -63,6 +61,4 @@ class TACHYON_EXPORT NodeCppConstructor : public base::CppConstructor {
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_CPP_CONSTRUCTOR_H_

tachyon/node/base/node_cpp_constructor_matcher.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_CPP_CONSTRUCTOR_MATCHER_H_
 #define TACHYON_NODE_BASE_NODE_CPP_CONSTRUCTOR_MATCHER_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include "third_party/node_addon_api/napi.h"
 
 #include "tachyon/base/binding/cpp_constructor_matcher.h"
@@ -44,6 +42,4 @@ class NodeCppConstructorMatcher : public base::CppConstructorMatcher {
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_CPP_CONSTRUCTOR_MATCHER_H_

tachyon/node/base/node_cpp_enum.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_CPP_ENUM_H_
 #define TACHYON_NODE_BASE_NODE_CPP_ENUM_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <string_view>
 #include <type_traits>
 
@@ -47,6 +45,4 @@ class NodeCppEnum : public NodeCppBindable {
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_CPP_ENUM_H_

tachyon/node/base/node_cpp_object.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_CPP_OBJECT_H_
 #define TACHYON_NODE_BASE_NODE_CPP_OBJECT_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <string_view>
 
 #include "tachyon/base/binding/cpp_stack_value.h"
@@ -154,6 +152,4 @@ Napi::FunctionReference NodeCppObject<T>::s_constructor_;
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_CPP_OBJECT_H_

tachyon/node/base/node_errors.cc

@@ -1,5 +1,3 @@
-#if defined(TACHYON_NODE_BINDING)
-
 #include "tachyon/node/base/node_errors.h"
 
 #include "absl/strings/substitute.h"
@@ -27,5 +25,3 @@ Napi::Error NoSuchConstructor(Napi::Env env) {
 }
 
 }  // namespace tachyon::node
-
-#endif  // defined(TACHYON_NODE_BINDING)

tachyon/node/base/node_errors.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_ERRORS_H_
 #define TACHYON_NODE_BASE_NODE_ERRORS_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <stddef.h>
 
 #include "third_party/node_addon_api/napi.h"
@@ -19,6 +17,4 @@ TACHYON_EXPORT Napi::Error NoSuchConstructor(Napi::Env env);
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_ERRORS_H_

tachyon/node/base/node_internals.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_INTERNALS_H_
 #define TACHYON_NODE_BASE_NODE_INTERNALS_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include <string>
 #include <string_view>
 #include <vector>
@@ -497,6 +495,4 @@ Napi::Value ToJSValue(const Napi::CallbackInfo& info, T&& value) {
 }  // namespace internal
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_INTERNALS_H_

tachyon/node/base/node_internals_forwards.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_INTERNALS_FORWARDS_H_
 #define TACHYON_NODE_BASE_NODE_INTERNALS_FORWARDS_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include "absl/meta/type_traits.h"
 #include "third_party/node_addon_api/napi.h"
 
@@ -51,6 +49,4 @@ struct HasToJSValue<
 }  // namespace internal
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_INTERNALS_FORWARDS_H_

tachyon/node/base/node_module.cc

@@ -1,5 +1,3 @@
-#if defined(TACHYON_NODE_BINDING)
-
 #include "tachyon/node/base/node_module.h"
 
 namespace tachyon::node {
@@ -24,5 +22,3 @@ NodeModule NodeModule::AddSubModule(std::string_view name) {
 }
 
 }  // namespace tachyon::node
-
-#endif  // defined(TACHYON_NODE_BINDING)

tachyon/node/base/node_module.h

@@ -1,8 +1,6 @@
 #ifndef TACHYON_NODE_BASE_NODE_MODULE_H_
 #define TACHYON_NODE_BASE_NODE_MODULE_H_
 
-#if defined(TACHYON_NODE_BINDING)
-
 #include "tachyon/export.h"
 #include "tachyon/node/base/node_cpp_bindable.h"
 #include "tachyon/node/base/node_cpp_callable.h"
@@ -56,6 +54,4 @@ class TACHYON_EXPORT NodeModule : public NodeCppBindable {
 
 }  // namespace tachyon::node
 
-#endif  // defined(TACHYON_NODE_BINDING)
-
 #endif  // TACHYON_NODE_BASE_NODE_MODULE_H_

tachyon/node/base/test/BUILD.bazel

@@ -1,10 +1,11 @@
+load("//bazel:tachyon.bzl", "if_node_binding")
 load("//bazel:tachyon_cc.bzl", "tachyon_cc_library")
 load("//bazel:tachyon_node.bzl", "tachyon_node_library")
 
 tachyon_cc_library(
     name = "color",
     testonly = True,
-    srcs = ["color.cc"],
+    srcs = if_node_binding(["color.cc"]),
     hdrs = ["color.h"],
     deps = [
         "//tachyon/base/binding/test:enum",
@@ -15,7 +16,7 @@ tachyon_cc_library(
 tachyon_cc_library(
     name = "point",
     testonly = True,
-    srcs = ["point.cc"],
+    srcs = if_node_binding(["point.cc"]),
     hdrs = ["point.h"],
     deps = [
         "//tachyon/base/binding/test:class",
@@ -26,7 +27,7 @@ tachyon_cc_library(
 tachyon_node_library(
     name = "test",
     testonly = True,
-    srcs = ["test.cc"],
+    srcs = if_node_binding(["test.cc"]),
     deps = [
         ":color",
         ":point",

tachyon/node/base/test/color.cc

@@ -1,11 +1,7 @@
-#if defined(TACHYON_NODE_BINDING)
-
 #include "tachyon/node/base/test/color.h"
 
 #include "tachyon/base/binding/test/color.h"
 
 void AddColor(tachyon::node::NodeModule& m) {
   using namespace tachyon::base::test;
 }
-
-#endif  // defined(TACHYON_NODE_BINDING)

tachyon/node/base/test/color.h

@@ -1,7 +1,3 @@
-#if defined(TACHYON_NODE_BINDING)
-
 #include "tachyon/node/base/node_module.h"
 
 void AddColor(tachyon::node::NodeModule& m);
-
-#endif  // defined(TACHYON_NODE_BINDING)

tachyon/node/base/test/enum.cc

@@ -0,0 +1,9 @@
+#include "tachyon/node/base/test/color.h"
+
+Napi::Object Init(Napi::Env env, Napi::Object exports) {
+  tachyon::node::NodeModule module(env, exports);
+  AddColor(module);
+  return exports;
+}
+
+NODE_API_MODULE(enum, Init)

tachyon/node/base/test/function.cc

@@ -0,0 +1,13 @@
+#include "tachyon/base/binding/test/functions.h"
+#include "tachyon/node/base/node_module.h"
+
+Napi::Object Init(Napi::Env env, Napi::Object exports) {
+  using namespace tachyon::base::test;
+  tachyon::node::NodeModule module(env, exports);
+  module.AddFunction("hello", &Hello)
+      .AddFunction("sum", &Sum, 1, 2)
+      .AddFunction("do_nothing", &DoNothing);
+  return exports;
+}
+
+NODE_API_MODULE(function, Init)

tachyon/node/base/test/point.cc

@@ -1,5 +1,3 @@
-#if defined(TACHYON_NODE_BINDING)
-
 #include "tachyon/node/base/test/point.h"
 
 void AddPoint(tachyon::node::NodeModule& m) {
@@ -29,5 +27,3 @@ void DoubleWithSharedPtr(std::shared_ptr<tachyon::base::test::Point> p) {
 }
 
 void DoubleWithUniquePtr(std::unique_ptr<tachyon::base::test::Point> p) {}
-
-#endif  // defined(TACHYON_NODE_BINDING)

tachyon/node/base/test/point.h

@@ -1,5 +1,3 @@
-#if defined(TACHYON_NODE_BINDING)
-
 #include <string_view>
 
 #include "tachyon/base/binding/test/point.h"
@@ -40,5 +38,3 @@ void DoubleWithReference(tachyon::base::test::Point& p);
 void DoubleWithSharedPtr(std::shared_ptr<tachyon::base::test::Point> p);
 
 void DoubleWithUniquePtr(std::unique_ptr<tachyon::base::test::Point> p);
-
-#endif  // defined(TACHYON_NODE_BINDING)