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

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

Tags

None

Referenced Files

	F154785302: D56622.id176415.diff
	Wed, Apr 29, 11:00 AM

	F154764199: D56622.id176415.diff
	Wed, Apr 29, 9:44 AM

	Unknown Object (File)
	Tue, Apr 28, 11:08 AM

	Unknown Object (File)
	Tue, Apr 28, 9:19 AM

	Unknown Object (File)
	Tue, Apr 28, 5:39 AM

	Unknown Object (File)
	Tue, Apr 28, 2:39 AM

	Unknown Object (File)
	Fri, Apr 24, 8:01 PM

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.Fri, Apr 24, 6:25 PM

imp created this revision.

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

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

imp mentioned this in D56621: sed(1): treat '}' as a command terminator in EMPTY, SUBST, and TR cases.Fri, Apr 24, 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