Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 16 Dec 2014 14:47:09 +0100
From:      Mathieu Arnold <mat@FreeBSD.org>
To:        Alex Kozlov <ak@FreeBSD.org>
Cc:        svn-ports-all@freebsd.org, svn-ports-head@freebsd.FreeBSD.ORG, ports-committers@freebsd.org
Subject:   Re: svn commit: r374800 - head/Mk/Uses
Message-ID:  <5B209A4BD7689CD74744CFA7@ogg.in.absolight.net>
In-Reply-To: <20141216133802.GA27647@ravenloft.kiev.ua>
References:  <201412161206.sBGC6fa5045717@svn.freebsd.org> <54902709.9090801@marino.st> <71EBD6FD6D89266B6651554A@ogg.in.absolight.net> <20141216133802.GA27647@ravenloft.kiev.ua>

next in thread | previous in thread | raw e-mail | index | archive | help
+--On 16 d=C3=A9cembre 2014 15:38:02 +0200 Alex Kozlov <ak@FreeBSD.org> =
wrote:
| On Tue, Dec 16, 2014 at 01:41:26PM +0100, Mathieu Arnold wrote:
|> +--On 16 d=C3=A9cembre 2014 13:35:21 +0100 John Marino
|> <freebsd.contact@marino.st> wrote:
|> | On 12/16/2014 13:06, Mathieu Arnold wrote:
|> |> Author: mat
|> |> Date: Tue Dec 16 12:06:40 2014
|> |> New Revision: 374800
|> |> URL: https://svnweb.freebsd.org/changeset/ports/374800
|> |> QAT: https://qat.redports.org/buildarchive/r374800/
|> |>=20
|> |> Log:
|> |>   ts=3D8, repeat after me, ts=3D8.
|> |>  =20
|> |>   With hat:	perl@
|> |>   Sponsored by:	Absolight
|> |>=20
|> |> Modified:
|> |>   head/Mk/Uses/perl5.mk
|> |>=20
|> |=20
|> | This is not just a perl thing.  search for "ts=3D4" and other 4-space
|> | tabs indicators in Mk, it's full of them (e.g. grep -nr "ts=3D4"
|> | /usr/ports/Mk).  If ts should be 8 across the board (and I think it
|> | should be), then all of these need to be changes and then documented.
|> Yes, it should, feel free to pitch in and provide patches to fix some
|> other files :-)
| Allow me to ask, why it should be 8 and not 4?

Because a tabulation is 8 spaces.  It should be consistent over the whole
ports tree.

--=20
Mathieu Arnold



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