Page MenuHomeFreeBSD

etcupdate: Make diff -l actually work
ClosedPublic

Authored by des on May 29 2026, 3:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Aug 19, 1:38 PM
Unknown Object (File)
Mon, Aug 17, 2:35 PM
Unknown Object (File)
Wed, Aug 12, 6:41 PM
Unknown Object (File)
Tue, Aug 11, 11:15 PM
Unknown Object (File)
Tue, Aug 11, 11:05 PM
Unknown Object (File)
Tue, Aug 11, 11:02 PM
Unknown Object (File)
Tue, Aug 11, 5:40 PM
Unknown Object (File)
Mon, Aug 10, 8:23 AM

Details

Summary

While here, remove unnecessary blank lines.

MFC after: 1 week
Fixes: 6d65c91b9a47 ("etcupdate: fix arguments order of diff command")

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 73547
Build 70430: arc lint + arc unit

Event Timeline

des requested review of this revision.May 29 2026, 3:34 PM

diff -l works for me as-is as intended:

>etcupdate diff -l

Changed: /etc/pam.d/ftpd

>etcupdate diff
Index: /etc/pam.d/ftpd
===================================================================
--- /etc/pam.d/ftpd (stock)
+++ /etc/pam.d/ftpd (local)
@@ -15,3 +15,4 @@

 # session
 session                required        pam_permit.so
+#XXX
\ No newline at end of file

Can you please provide an example where it fails to produce output needed?

# etcupdate diff -l | grep \^Changed
Changed: /etc/master.passwd
# etcupdate diff | grep \^Index     
Index: /etc/master.passwd
Index: /etc/pam.d/ftpd
Index: /etc/sysctl.conf
Index: /etc/group
Index: /etc/shells
This revision is now accepted and ready to land.Jun 3 2026, 10:59 AM
This revision was automatically updated to reflect the committed changes.