Date: Sat, 30 Apr 2016 10:58:46 +0200 From: =?UTF-8?Q?Ren=C3=A9_Ladan?= <rene@freebsd.org> To: Alexey Dokuchaev <danfe@freebsd.org> Cc: svn-ports-all@freebsd.org, Christoph Moench-Tegeder <cmt@freebsd.org>, ports-committers@freebsd.org, Christoph Moench-Tegeder <cmt@burggraben.net>, svn-ports-head@freebsd.org Subject: Re: svn commit: r414106 - head/graphics/shotwell Message-ID: <CADL2u4gj4dui02ebYpQ%2B3FzB440e4cSwWSzoyu5dhjnBr%2BdysA@mail.gmail.com> In-Reply-To: <20160429074724.GA63859@FreeBSD.org> References: <201604271242.u3RCg7IB031266@repo.freebsd.org> <20160429071040.GA56016@FreeBSD.org> <20160429074042.GA1547@elch.exwg.net> <20160429074724.GA63859@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Op 29 apr. 2016 9:47 a.m. schreef "Alexey Dokuchaev" <danfe@freebsd.org>: > > On Fri, Apr 29, 2016 at 09:40:42AM +0200, Christoph Moench-Tegeder wrote: > > So - should I fix that at the next update or only when I have to touch > > that line anyway? > > I would leave it as is now, it does not warrant another commit just for > its sake. Next time USES gains or loses an entry, you should take this > opportunity to sort and properly wrap it. > I should be the one to blame here, not Christoph, there is an "Approved by" line in the commit message. And I should count line lengths next time I review a patch. Regards, Ren=C3=A9 https://rene-ladan.nl/
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADL2u4gj4dui02ebYpQ%2B3FzB440e4cSwWSzoyu5dhjnBr%2BdysA>