Skip to content

switch version parsing to use standard commands - #4347

Merged
sxa merged 3 commits into
adoptium:masterfrom
sxa:fix_proprietary_grep
Jan 9, 2026
Merged

sxa merged 3 commits into
adoptium:masterfrom
sxa:fix_proprietary_grep

Conversation

@sxa

@sxa sxa commented Dec 22, 2025

Copy link
Copy Markdown
Member

grep -Eo is not standard grep syntax and is incompatible with Solaris producing errors when this is run. The incompatible code was introduced in #4304.

Change to using awk as it avoids the fork of additional processes to parse the files and is standard across UNIX-based systems.

@judovana has suggested using a simpler grep construct but personally I'm losing interest in testing things on this platform at the moment but happy if someone wishes to propose and test an alternative ;-)

Signed-off-by: Stewart X Addison <sxa@ibm.com>
@sxa sxa self-assigned this Dec 22, 2025
@github-actions github-actions Bot added compatibility Issues that relate to how our code works with other third party code bases testing Issues that enhance or fix our test suites labels Dec 22, 2025

@karianna karianna left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@sxa Probably best to check my suggestions carefully.

Comment thread sbin/build.sh Outdated
Comment thread sbin/build.sh
Comment thread sbin/build.sh Outdated
Comment thread sbin/build.sh
Co-authored-by: Martijn Verburg <martijnverburg@gmail.com>
Co-authored-by: Martijn Verburg <martijnverburg@gmail.com>
@sxa
sxa requested a review from adamfarley January 9, 2026 09:29

@adamfarley adamfarley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@sxa
sxa merged commit 61d137a into adoptium:master Jan 9, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compatibility Issues that relate to how our code works with other third party code bases testing Issues that enhance or fix our test suites

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants