Skip to content

fix(install): support older awk checksum parsing - #28784

Merged
fcoury-oai merged 1 commit into
mainfrom
codex/fix-mawk-installer-checksum
Jun 18, 2026
Merged

fix(install): support older awk checksum parsing#28784
fcoury-oai merged 1 commit into
mainfrom
codex/fix-mawk-installer-checksum

Conversation

@fcoury-oai

Copy link
Copy Markdown
Contributor

Why

The standalone installer validates package checksums with an awk interval expression. Older mawk releases do not support that expression, so they reject valid 64-character digests and report that the release manifest is missing an entry. This affects both x64 and ARM64 systems on common Debian-derived environments.

Fixes #24219.

What Changed

Replace the awk interval expression with an explicit length check plus rejection of non-hexadecimal characters. This preserves the existing SHA-256 validation and lowercase normalization while working with older awk implementations.

How to Test

  1. Build and run the checksum predicate with mawk 1.3.4 20121129.
  2. Confirm the old interval predicate rejects a valid 64-character digest.
  3. Confirm the updated predicate accepts that digest.
  4. Put the old mawk binary first on PATH as awk and run scripts/install/install.sh with an isolated HOME, CODEX_HOME, and CODEX_INSTALL_DIR.
  5. Confirm Codex installs successfully and the installed binary reports version 0.140.0.
  6. Verify the predicate rejects wrong-length digests, non-hexadecimal digests, and entries for another asset while accepting uppercase hexadecimal digests.

@fcoury-oai
fcoury-oai force-pushed the codex/fix-mawk-installer-checksum branch from cc359f4 to dc47ba9 Compare June 18, 2026 00:45
@fcoury-oai
fcoury-oai merged commit f22d15b into main Jun 18, 2026
31 checks passed
@fcoury-oai
fcoury-oai deleted the codex/fix-mawk-installer-checksum branch June 18, 2026 02:12
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

install.sh checksum lookup fails on systems using mawk

2 participants