Page MenuHomeFreeBSD

etcupdate: remove redundant diff3 flag
ClosedPublic

Authored by thj on Mar 25 2022, 4:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 26, 10:53 AM
Unknown Object (File)
Sat, Nov 8, 8:07 AM
Unknown Object (File)
Thu, Nov 6, 6:49 AM
Unknown Object (File)
Thu, Nov 6, 6:02 AM
Unknown Object (File)
Sun, Nov 2, 7:29 PM
Unknown Object (File)
Oct 19 2025, 12:11 PM
Unknown Object (File)
Oct 19 2025, 12:10 PM
Unknown Object (File)
Oct 18 2025, 10:01 PM
Subscribers

Details

Reviewers
bapt
pstef
0mp
Group Reviewers
Klara
Summary

With diff3 -A and -m select different and conflicting output modes. When
both are passed -m is used by gnu diff3. Remove the extra -A flag.

Test Plan

The reduntant behaviour can be demonstrated by:

$ cd /usr/src/usr.bin/diff3/tests/
$ diff3 -m -A -L 1.txt -L 2.txt -L 3.txt 1.txt 2.txt 3.txt  > one
$ diff3 -m -L 1.txt -L 2.txt -L 3.txt 1.txt 2.txt 3.txt  > two   
$ diff one two

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44891
Build 41779: arc lint + arc unit

Event Timeline

thj requested review of this revision.Mar 25 2022, 4:20 PM

I don't use etcupdate yet, but this change looks reasonable to me.

0mp added a subscriber: 0mp.

I run your test case and it seems like -A can safely be removed.

Note: I think there is something not right with the grammar in the summary. Otherwise, it seems to be ready to be committed. An MFC would be nice as well.

This revision is now accepted and ready to land.Mar 25 2022, 5:51 PM