Skip to content

Update sed-version to 4.10 from 4.9#18

Open
zosopentoolsmain wants to merge 1 commit into
mainfrom
bump-sed-version-4.10
Open

Update sed-version to 4.10 from 4.9#18
zosopentoolsmain wants to merge 1 commit into
mainfrom
bump-sed-version-4.10

Conversation

@zosopentoolsmain
Copy link
Copy Markdown

No description provided.

@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented Apr 22, 2026

🤖 Augment PR Summary

Summary: Bumps the GNU sed version used by this port from 4.9 to 4.10.
Why: Keeps the build targeting the latest upstream sed release by updating the version variable used to form the download URL.

🤖 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. 2 suggestions posted.

Fix All in Augment

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

Comment thread buildenv
@@ -1,5 +1,5 @@
# bump: sed-version /SED_VERSION="(.*)"/ https://ftp.gnu.org/gnu/sed/|re:/href="sed-([\d.]+).tar.gz"/$1/|semver:*
SED_VERSION="4.9"
SED_VERSION="4.10"
Copy link
Copy Markdown

@augmentcode augmentcode Bot Apr 22, 2026

Choose a reason for hiding this comment

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

SED_VERSION is bumped to 4.10, but zopen_check_results() still includes a “Testsuite summary for GNU sed 4.9” block and expectedFailures:9, which now looks stale and could cause CI/result checks to be misleading for 4.10.

Severity: medium

Fix This in Augment

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

Comment thread buildenv
@@ -1,5 +1,5 @@
# bump: sed-version /SED_VERSION="(.*)"/ https://ftp.gnu.org/gnu/sed/|re:/href="sed-([\d.]+).tar.gz"/$1/|semver:*
SED_VERSION="4.9"
SED_VERSION="4.10"
Copy link
Copy Markdown

@augmentcode augmentcode Bot Apr 22, 2026

Choose a reason for hiding this comment

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

With the sed version bump, please double-check that stable-patches/xmalloc.c.patch (which references sed-4.9/... in its file headers) still applies cleanly against the 4.10 sources in your patch-application flow.

Severity: medium

Fix This in Augment

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

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.

1 participant