cvs commit: ports/archivers/lzma Makefile ports/archivers/paq Makefile ports/archivers/ppmd-7z Makefile ports/devel/abi-compliance-checker Makefile ports/devel/api-sanity-autotest Makefile ports/devel/distorm Makefile ports/devel/libdisasm ...

[Alexey Dokuchaev]

On Thu, Jul 08, 2010 at 01:29:04PM +0000, b. f. wrote:
> On 7/8/10, Erwin Lansing <erwin at freebsd.org> wrote:
> > On Thu, Jul 08, 2010 at 12:51:38PM +0000, Brendan Fabeny wrote:
> >>   Log:
> >>   Change my ports to use my FreeBSD.org email address.
> >
> > This is a sweeping commit explicitly not allowed during a slush.  Also,
> > I am not seeing approval by one of your mentors.  Please take more care.
> 
> Sorry, I'll do so.  I forgot to mention in the commit message that
> makc@ gave his approval, and miwi@ did not object to the commit.  And
> my reading of the criteria for sweeping change suggested that this
> would be admissible because it was a "cosmetic change", mostly to leaf
> ports (with the exception of textproc/wv). If I misunderstood what is
> considered to be cosmetic, please let me know.

Sweeping commit is any commit that touches more than a couple of ports,
and it can even be more strict than that (e.g. shlib port update which
potentially can break things).  Also please see RFC 1855 about excessive
quoting in email.

./danfe