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

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

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

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

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

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

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

	Unknown Object (File)
	May 17 2026, 8:14 AM

	Unknown Object (File)
	May 13 2026, 11:46 PM

View All 28 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