Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 5 Dec 2017 19:19:33 -0800
From:      Conrad Meyer <cem@freebsd.org>
To:        "Rodney W. Grimes" <rgrimes@freebsd.org>
Cc:        Eitan Adler <eadler@freebsd.org>, src-committers <src-committers@freebsd.org>,  svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r326614 - in head: etc/mtree usr.bin usr.bin/sponge usr.bin/tee
Message-ID:  <CAG6CVpUm5KZYox_yNgg59ZiAM45rKp-69PnpdJTCJt6PdMO8_Q@mail.gmail.com>
In-Reply-To: <201712060310.vB63A1Bn054608@pdx.rh.CN85.dnsmgr.net>
References:  <201712060247.vB62lksa053161@repo.freebsd.org> <201712060310.vB63A1Bn054608@pdx.rh.CN85.dnsmgr.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Dec 5, 2017 at 7:10 PM, Rodney W. Grimes
<freebsd@pdx.rh.cn85.dnsmgr.net> wrote:
> [ Charset UTF-8 unsupported, converting... ]
>> Author: eadler
>> Date: Wed Dec  6 02:47:46 2017
>> New Revision: 326614
>> URL: https://svnweb.freebsd.org/changeset/base/326614
>>
>> Log:
>>   sponge(1): revert
>>
>>   I did a complete buildworld and test... with the program disconnected
>>   from the tree. Revert the change for now.
>>
>>   (this keeps the change to .arclint which is still correct)
>
> Commited with no log entry that makes any sense about why it was changed,
> please revert .arclint and recommit with a proper log entry for .arclint
> as to why all /tests/ dirs should be ignored.

I have to agree with Rod.  The commit message for the arclint change
is currently "sponge(1): initial commit =E2=80=A6" and doesn't say anything
about the arclint change.  Even if the change is correct, the commit
message isn't.

Conrad



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG6CVpUm5KZYox_yNgg59ZiAM45rKp-69PnpdJTCJt6PdMO8_Q>