Page MenuHomeFreeBSD

sed: add deprecation notice for -i .bak
Needs ReviewPublic

Authored by emaste on Mar 8 2021, 12:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 31, 7:07 AM
Unknown Object (File)
Sat, Jan 24, 6:11 PM
Unknown Object (File)
Nov 18 2025, 3:06 PM
Unknown Object (File)
Nov 16 2025, 11:33 PM
Unknown Object (File)
Nov 16 2025, 8:54 PM
Unknown Object (File)
Nov 16 2025, 7:06 PM
Unknown Object (File)
Nov 16 2025, 2:55 PM
Unknown Object (File)
Nov 10 2025, 12:19 PM
Subscribers

Details

Reviewers
kevans
tobik
Summary

PR: 254091

Test Plan

Needs test suite update, posted for discussion.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste created this revision.
usr.bin/sed/main.c
142

I think the warning sounds like an error and it is not very clear that sed actually goes ahead and changes the file anyway.

val_packett.cool added inline comments.
usr.bin/sed/main.c
138

Hmm. Still considering -i '' non-deprecated — would that really facilitate a transition to the style everyone else uses?