sed(1): also treat '}' as a terminator in compile_flags; add tests
Needs ReviewPublic
Actions

Authored by imp on Apr 24 2026, 6:25 PM.

Tags

None

Referenced Files

	F163118782: D56622.diff
	Mon, Jul 20, 5:53 AM

	Unknown Object (File)
	Tue, Jul 14, 11:49 AM

	Unknown Object (File)
	Mon, Jun 29, 10:05 AM

	Unknown Object (File)
	Jun 17 2026, 12:49 PM

	Unknown Object (File)
	Jun 12 2026, 2:56 AM

	Unknown Object (File)
	Jun 10 2026, 6:09 AM

	Unknown Object (File)
	May 31 2026, 2:01 AM

	Unknown Object (File)
	May 17 2026, 12:08 PM

View All 31 Files

Subscribers

None

This revision needs review, but there are no reviewers specified.

Details

Reviewers: None

Summary

Add ATF tests covering '}' as a command terminator for the EMPTY (p),
SUBST (s), and TR (y) command types.

Assisted-by: Claude (Opus 4.6 1M context)
Sponsored by: Netflix

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 72526
Build 69409: arc lint + arc unit

Event Timeline

imp requested review of this revision.Apr 24 2026, 6:25 PM

imp created this revision.

Harbormaster completed remote builds in B72526: Diff 176415.Apr 24 2026, 6:25 PM

imp added a parent revision: D56621: sed(1): treat '}' as a command terminator in EMPTY, SUBST, and TR cases.Apr 24 2026, 6:25 PM

imp mentioned this in D56621: sed(1): treat '}' as a command terminator in EMPTY, SUBST, and TR cases.Apr 24 2026, 6:42 PM

Revision Contents
Changeset List

Path

Size

usr.bin/

sed/

tests/

sed2_test.sh

41 lines

Diff 176415

View Options

usr.bin/sed/tests/sed2_test.sh

sed(1): also treat '}' as a terminator in compile_flags; add testsNeeds ReviewPublicActions