Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 3 Apr 2015 15:17:46 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Sunpoet Po-Chuan Hsieh <sunpoet@freebsd.org>
Cc:        svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, Bartek Rutkowski <robak@freebsd.org>, ports-committers@freebsd.org
Subject:   Re: svn commit: r383112 - head/databases/postgresql94-server
Message-ID:  <20150403151746.GA13396@FreeBSD.org>
In-Reply-To: <CAMHz58SX8rGX76YHonZSaK0BUSuExHS9h0x3hykZka5aWhh45A@mail.gmail.com>
References:  <201504031232.t33CW7wi035371@svn.freebsd.org> <CAMHz58SX8rGX76YHonZSaK0BUSuExHS9h0x3hykZka5aWhh45A@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Apr 03, 2015 at 11:08:47PM +0800, Sunpoet Po-Chuan Hsieh wrote:
> Regarding the CPE PRs, though Shun submitted the patch as "USES+=<tab>cpe",
> I would be grateful if you could use more tabs to have same indent with
> other lines. It needs 2 tabs in this case.

+1000.  I was hoping someone would finally speak up on this matter (so I
won't be blamed for nitpicking this time ;-)

Also, in addition to bogus indentation, I don't see any need to use += vs.
just = .

./danfe



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