Enable bindgen tests for windows

bazelbuild · Jan 29, 2025 · 7615691 · 7615691
1 parent 7584b08
commit 7615691
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 18 deletions.
diff --git a/.bazelci/presubmit.yml b/.bazelci/presubmit.yml
@@ -918,20 +918,18 @@ tasks:
       - "//..."
     test_targets:
       - "//..."
-  # # TODO: https://github.com/bazelbuild/rules_rust/issues/2009
-  # # The bindgen rules are currently broken on windows
-  # extensions_bindgen_windows:
-  #   platform: windows
-  #   name: Extensions Bindgen
-  #   working_directory: extensions/bindgen
-  #   build_flags: *aspects_flags
-  #   test_flags: *aspects_flags
-  #   build_targets:
-  #     - "--"
-  #     - "//..."
-  #   test_targets:
-  #     - "--"
-  #     - "//..."
+  extensions_bindgen_windows:
+    platform: windows
+    name: Extensions Bindgen
+    working_directory: extensions/bindgen
+    build_flags: *aspects_flags
+    test_flags: *aspects_flags
+    build_targets:
+      - "--"
+      - "//..."
+    test_targets:
+      - "--"
+      - "//..."
   extensions_mdbook_linux:
     platform: ubuntu2004
     name: Extensions MdBook

diff --git a/.bcr/extensions/bindgen/presubmit.yml b/.bcr/extensions/bindgen/presubmit.yml
@@ -4,8 +4,7 @@ bcr_test_module:
     platform:
       - "macos_arm64"
       - "ubuntu2004"
-      # # TODO: https://github.com/bazelbuild/rules_rust/issues/2009
-      # - "windows"
+      - "windows"
     bazel: ["7.x"]
   tasks:
     run_tests:

diff --git a/extensions/bindgen/private/bindgen.bzl b/extensions/bindgen/private/bindgen.bzl
@@ -130,7 +130,7 @@ def _get_user_link_flags(cc_lib):
     return linker_flags
 
 def _generate_cc_link_build_info(ctx, cc_lib):
-    """Produce the eqivilant cargo_build_script providers for use in linking the library.
+    """Produce the equivalent cargo_build_script providers for use in linking the library.
 
     Args:
         ctx (ctx): The rule's context object
@@ -391,7 +391,10 @@ rust_bindgen = rule(
             default = True,
         ),
         "wrap_static_fns": attr.bool(
-            doc = "Whether to create a separate .c file for static fns. Requires nightly toolchain, and a header that actually needs this feature (otherwise bindgen won't generate the file and Bazel complains).",
+            doc = (
+                "Whether to create a separate .c file for static functions. Requires nightly toolchain, " +
+                "and a header that actually needs this feature (otherwise bindgen won't generate the file and Bazel complains)."
+            ),
             default = False,
         ),
         "_process_wrapper": attr.label(