On Wed, Nov 19, 2014 at 05:11:14PM +0300, Anton Yuzhaninov wrote: A> On 11/19/14 16:57, Mark Felder wrote: A> > New Revision: 274710 A> > URL: https://svnweb.freebsd.org/changeset/base/274710 A> > A> > Log: A> > Expose groups by default in ifconfig output. This was never hidden by A> > OpenBSD; unsure why we chose to do so. As groups are a requirement for A> > pf, exposing them by default will make our pf implementation less A> > confusing. A> ... A> > A> > Modified: head/sbin/ifconfig/ifgroup.c A> > ============================================================================== A> > --- head/sbin/ifconfig/ifgroup.c Wed Nov 19 13:31:08 2014 (r274709) A> > +++ head/sbin/ifconfig/ifgroup.c Wed Nov 19 13:57:39 2014 (r274710) A> > @@ -86,9 +86,6 @@ getifgroups(int s) A> > struct ifgroupreq ifgr; A> > struct ifg_req *ifg; A> > A> > - if (!verbose) A> > - return; A> A> Please return groups output under verbose flag. A> A> pf is used not by all. A> A> And more longer ifconfig output is unhandy. Especially on hosts with many A> interfaces. A> A> Better to document ifconfig -v in pf.conf(5)
Better to document it in ifconfig(8). Actually ifgroups are not a part of pf(5). And commit has nothing to do with "our pf implementation being confusing". P.S. I was pretty sure that users would not like changing defaults for ifconfig. -- Totus tuus, Glebius. _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
