cvs commit: ports/sysutils/nvclock/files patch-src::Makefile.in

Alexey Dokuchaev danfe at FreeBSD.org
Sat Sep 11 10:07:55 UTC 2010


On Sat, Sep 11, 2010 at 11:55:50AM +0200, Remko Lodder wrote:
> On Sep 11, 2010, at 11:51 AM, Alexey Dokuchaev wrote:
> > On Sat, Sep 11, 2010 at 11:33:20AM +0200, Remko Lodder wrote:
> >> We refer to this as
> >> 
> >> PR:	<category>/<number>
> > 
> > In most cases, yes, absolutely, but:
> > 
> > The committed patch was not from the PR; submitter suggested marking the
> > port as jobs-unsafe (and I hate marking ports unsafe or broken instead of
> > properly fixing them).  Technically, I could have omitted any attribution
> > whatsoever, but decided to do it nonetheless (making clear that commit
> > merely closes the PR, not taking any code/idea from it).  Thanks for
> > asking though.
> 
> no, when refering to a PR we always do that.
> 
> We can tell in the changelog itself that the PR had not been used itself.

Frankly speaking, I was reluctant to use non-canonical tag form, as much
as I hate doing it.  I spent about several minutes thinking of proper and
clear text for explanation of the role of the PR, but simple "make jobs
unsafeness report" just occurred to me this very moment.  :-(

Sorry I did not put enough thinking into it.

./danfe


More information about the cvs-all mailing list