Skip to content

[maintenance] Windows CI#1439

Merged
restingbull merged 6 commits intobazelbuild:masterfrom
agluszak:windows-ci
Feb 10, 2026
Merged

[maintenance] Windows CI#1439
restingbull merged 6 commits intobazelbuild:masterfrom
agluszak:windows-ci

Conversation

@agluszak
Copy link
Contributor

@agluszak agluszak commented Jan 4, 2026 •
edited
Loading

This change enables Windows builds and tests in the CI pipeline and fixes
various Windows compatibility issues throughout the codebase.

Platform-specific path handling:

  • Replace /dev/null with ctx.actions.declare_file for null outputs on Windows
  • Use platform-appropriate path separators in classpath construction
  • Fix bzlmod classpath resolution for Kotlin compiler on Windows

Ktlint fixes:

  • Simplify ktlint_fix and ktlint_test implementations
  • Use proper argument files instead of shell-specific constructs
  • Remove windows.bzl utility in favor of inline platform checks

Test infrastructure:

  • Convert integration test runner to java_binary for cross-platform support
  • Fix temporary file handling in tests for Windows compatibility
  • Update test assertions to handle platform-specific path formats

CI configuration:

  • Enable Windows builds in presubmit.yml
  • Add Windows-specific bazelrc configurations
  • Update MODULE.bazel with Windows toolchain support

@agluszak agluszak force-pushed the windows-ci branch 3 times, most recently from 0e1d972 to 3066829 Compare January 6, 2026 00:56
Bencodes
Bencodes reviewed Jan 6, 2026
View reviewed changes
kotlin/internal/jvm/impl.bzl
executable_name = executable_name + ".exe"
return ctx.actions.declare_file(executable_name)

def _create_windows_exe_launcher(ctx, executable, java_executable, classpath, main_class, jvm_flags_for_launcher, runfiles_enabled, coverage_main_class = None):
Copy link
Collaborator

@Bencodes Bencodes Jan 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is most of this impl. just copied over from the java rules? https://github.com/bazelbuild/rules_java/blob/aa0345c749ae24b4ff3e9ca92e52bbcd96bc135a/java/bazel/rules/bazel_java_binary.bzl#L280-L304

Copy link
Contributor Author

@agluszak agluszak Jan 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup

@agluszak agluszak marked this pull request as ready for review January 9, 2026 12:18
@Bencodes
Copy link
Collaborator

Bencodes commented Jan 23, 2026

@agluszak this one has a bunch of merge conflicts in it, can you take a look at resolving these?

@agluszak
Copy link
Contributor Author

agluszak commented Jan 30, 2026

@Bencodes merged master!

Bencodes
Bencodes reviewed Jan 31, 2026
View reviewed changes
restingbull
restingbull reviewed Feb 2, 2026
View reviewed changes
@agluszak agluszak force-pushed the windows-ci branch 2 times, most recently from 609929f to 56ade49 Compare February 7, 2026 16:13
@agluszak
Copy link
Contributor Author

agluszak commented Feb 7, 2026

GitHub seems to be having a hard time now :/

(21:08:51) WARNING: Download from https://github.com/bazelbuild/rules_pkg/releases/download/1.0.1/rules_pkg-1.0.1.tar.gz failed: class java.io.IOException GET returned 502 Bad Gateway or Proxy Error
(21:08:51) INFO: Repository rules_pkg+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  C:/b/hejvqjrb/external/bazel_tools/tools/build_defs/repo/http.bzl:431:31: in <toplevel>
(21:08:51) ERROR: C:/b/hejvqjrb/external/bazel_tools/tools/build_defs/repo/http.bzl:155:45: An error occurred during the fetch of repository 'rules_pkg+':
   Traceback (most recent call last):
	File "C:/b/hejvqjrb/external/bazel_tools/tools/build_defs/repo/http.bzl", line 155, column 45, in _http_archive_impl
		download_info = ctx.download_and_extract(
Error in download_and_extract: java.io.IOException: Error downloading [https://github.com/bazelbuild/rules_pkg/releases/download/1.0.1/rules_pkg-1.0.1.tar.gz] to C:/b/hejvqjrb/external/rules_pkg+/temp13767907055791222701/rules_pkg-1.0.1.tar.gz: GET returned 502 Bad Gateway or Proxy Error
(21:08:51) WARNING: Target pattern parsing failed.
(21:08:51) ERROR: Skipping '//src/...': error loading package under directory 'src': no such package '@@rules_pkg+//': java.io.IOException: Error downloading [https://github.com/bazelbuild/rules_pkg/releases/download/1.0.1/rules_pkg-1.0.1.tar.gz] to C:/b/hejvqjrb/external/rules_pkg+/temp13767907055791222701/rules_pkg-1.0.1.tar.gz: GET returned 502 Bad Gateway or Proxy Error
(21:08:51) ERROR: error loading package under directory 'src': no such package '@@rules_pkg+//': java.io.IOException: Error downloading [https://github.com/bazelbuild/rules_pkg/releases/download/1.0.1/rules_pkg-1.0.1.tar.gz] to C:/b/hejvqjrb/external/rules_pkg+/temp13767907055791222701/rules_pkg-1.0.1.tar.gz: GET returned 502 Bad Gateway or Proxy Error

@agluszak
Copy link
Contributor Author

agluszak commented Feb 9, 2026 •
edited
Loading

@Bencodes could you re-run the CI? It should pass

@restingbull restingbull merged commit a874935 into bazelbuild:master Feb 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@restingbull restingbull restingbull approved these changes

@ahumesky ahumesky Awaiting requested review from ahumesky ahumesky is a code owner

@cgruber cgruber Awaiting requested review from cgruber cgruber is a code owner

@nkoroste nkoroste Awaiting requested review from nkoroste nkoroste is a code owner

@Bencodes Bencodes Awaiting requested review from Bencodes Bencodes is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@agluszak @Bencodes @restingbull