Skip to content

Fix compile-time issues for Bazel on z/OS - #2

Open
RohiniShankari wants to merge 9 commits into
zopencommunity:mainfrom
RohiniShankari:main
Open

Fix compile-time issues for Bazel on z/OS#2
RohiniShankari wants to merge 9 commits into
zopencommunity:mainfrom
RohiniShankari:main

Conversation

@RohiniShankari

Copy link
Copy Markdown
Collaborator

Summary

  • Fixes compile-time issues encountered while building Bazel on z/OS.
  • removed unnecessary files

Notes

  • The runtime OpenJ9 garbage collector issue is not addressed in this PR.
  • I am still investigating the appropriate location to patch the GC detection logic, as the relevant source appears to be generated during the build process.

@augmentcode

augmentcode Bot commented Jul 7, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Review Notes (unanchored due to PR size/API limits):

  • I attempted to add inline review comments for buildenv, README.md, CODEOWNERS, and .bazelversion, but GitHub rejected the anchors (likely related to very large PR file-list truncation).
  • buildenv: hard-coded /data/zopen/usr/local/... PATH prefixes can reduce portability across z/OS installations with different prefixes.
  • buildenv: EXTRA_BAZEL_ARGS is exported as a multiline string with a trailing space and overwrites any previously-set value, which can lead to subtle argument-parsing issues.
  • buildenv: setting BAZEL_JAVAC_OPTS unconditionally may override existing build-harness options.
  • CODEOWNERS: switching to @bazelbuild/triage may be ignored if external org teams aren�t supported for CODEOWNERS in this repo.
  • README.md: replacing port-specific instructions with upstream Bazel README may confuse users of this zopen/z/OS port.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread .bazelci/.bazelversion
@@ -1 +0,0 @@
7.1.1

@augmentcode augmentcode Bot Jul 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

.bazelci/.bazelversion Removing the pinned Bazel version may make builds less reproducible for contributors/tools that rely on bazelisk/.bazelversion pinning. If the intent is to drop CI/dev scaffolding, consider confirming there�[39m�[0m�[0m�[0m�[39m�[0m isn�[39m�[0m�[0m�[0m�[39m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[0m�[0m�[39m�[0m�[39m�[0m is an alternative pin/versioning mechanism for the z/OS port build.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

Implemented:
- Added AutoServiceProcessor to BAZEL_JAVAC_OPTS in buildenv to resolve the unsupported charset error.
- Updated unix_cc_configure.bzl to execute scripts through /bin/bash.
- Added z/OS enum support in Os.java.

Work in progress:
- Local rules_python and rules_go overrides.
- Initial z/OS support changes in rules_python and rules_go.
- working on rules_python and rules_go
RohiniShankari and others added 6 commits July 28, 2026 20:31
Updated the README to enhance clarity and structure for building Bazel on z/OS, including detailed steps for cloning the repository, tagging binary files, configuring toolchains, and building.

Signed-off-by: RohiniShankari <136949412+RohiniShankari@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants