Skip to content

Upgrade version from 9.10 to 9.11#112

Merged
sachintu47 merged 1 commit intozopencommunity:mainfrom
sabi789:main
Apr 30, 2026
Merged

Upgrade version from 9.10 to 9.11#112
sachintu47 merged 1 commit intozopencommunity:mainfrom
sabi789:main

Conversation

@sabi789
Copy link
Copy Markdown
Member

@sabi789 sabi789 commented Apr 28, 2026

No description provided.

@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented Apr 28, 2026

🤖 Augment PR Summary

Summary: Upgrades this port to GNU coreutils 9.11 and refreshes the bundled stable patches to apply cleanly on the new upstream release.

Changes:

  • Bumped COREUTILS_VERSION from 9.10 to 9.11
  • Updated z/OS (MVS) patch for lib/nproc.c to use a platform CPU-count API
  • Rebased the z/OS-specific cat patch hunk offsets/contexts for the new upstream file
  • Adjusted the z/OS-specific pinky patch around pw_gecos handling

Technical Notes: This PR is primarily a version bump plus patch re-alignment to upstream 9.11.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

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.

+ // Validate the program codeset
+ if (strcmp(info->pgmcodeset, "ISO8859-1") != 0 && strcmp(info->pgmcodeset, "819") != 0) {
+ error(EXIT_FAILURE, EINVAL, _("Only pgmcodeset ISO8859-1 is supported"));
+ error(EXIT_FAILURE, EINVAL, _("Only pgmcodeset ISO8859-1 is supported"));
Copy link
Copy Markdown

@augmentcode augmentcode Bot Apr 28, 2026

Choose a reason for hiding this comment

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

stable-patches/src/cat.c.patch:150: The validation allows pgmcodeset values ISO8859-1 and 819, but the error message says only ISO8859-1 is supported, which can mislead users when 819 is rejected/accepted.

Severity: low

Fix This in Augment

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

Signed-off-by: sabi789 <sabithac6298@gmail.com>
Copy link
Copy Markdown
Member

@sachintu47 sachintu47 left a comment

Choose a reason for hiding this comment

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

LGTM

@sachintu47 sachintu47 merged commit 5a06488 into zopencommunity:main Apr 30, 2026
2 checks passed
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