svn commit: r326039 - in head: etc/mtree usr.bin/head usr.bin/head/tests

[Alan Somers]

On Mon, Nov 20, 2017 at 6:32 PM, Ed Maste <emaste at freebsd.org> wrote:
> On 20 November 2017 at 17:55, Alan Somers <asomers at freebsd.org> wrote:
>> Author: asomers
>> Date: Mon Nov 20 22:55:02 2017
>> New Revision: 326039
>> URL: https://svnweb.freebsd.org/changeset/base/326039
>>
>> Log:
>>   Add ATF tests for head(1)
>>
>>   Submitted by: Fred Schlecter <https://github.com/fjs-github>
>>   Reviewed by:  asomers, jilles
>>   MFC after:    3 weeks
>>   Differential Revision:        https://github.com/freebsd/freebsd/pull/127
>
> It's certainly a good idea to capture the source of changes that come
> in via GitHub pull requests, but the 'Differential Revision' field is
> specifically for reviews in our own hosted Phabricator instance.
>
> We don't have an officially established metadata field for changes
> that come in via pull requests at the moment. I'd suggest we just go
> with:
> Pull request: https://github.com/freebsd/freebsd/pull/127

Are there any automated tools that care about the "Differential
Revision" field?  If so, they should be able to tell the difference
between GitHub revisions and Phabricator revisions based on the URL.
Using "Differential Revision" instead of "Pull request" is handy,
because it's part of the standard commit template, which encourages
consistency.

-Alan