From owner-cvs-all  Thu Sep 13 20: 1:56 2001
Delivered-To: cvs-all@freebsd.org
Received: from tao.org.uk (genius.tao.org.uk [212.135.162.51])
	by hub.freebsd.org (Postfix) with ESMTP
	id 6511737B408; Thu, 13 Sep 2001 20:01:49 -0700 (PDT)
Received: by tao.org.uk (Postfix, from userid 100)
	id D24EF6B; Fri, 14 Sep 2001 04:01:46 +0100 (BST)
Date: Fri, 14 Sep 2001 04:01:46 +0100
From: Josef Karthauser <joe@tao.org.uk>
To: Garance A Drosihn <drosih@rpi.edu>
Cc: Brian Somers <brian@freebsd-services.com>,
	cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: Re: Proposed patch to log_accum.pl
Message-ID: <20010914040146.A32158@tao.org.uk>
References: <joe@tao.org.uk> <200109140151.f8E1prd00702@hak.lan.Awfulhak.org> <20010914025459.E29860@tao.org.uk> <p0510100db7c71a45e2a7@[128.113.24.47]>
Mime-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-md5;
	protocol="application/pgp-signature"; boundary="82I3+IH0IqGh5yIs"
Content-Disposition: inline
User-Agent: Mutt/1.2.5i
In-Reply-To: <p0510100db7c71a45e2a7@[128.113.24.47]>; from drosih@rpi.edu on Thu, Sep 13, 2001 at 10:32:00PM -0400
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG


--82I3+IH0IqGh5yIs
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Sep 13, 2001 at 10:32:00PM -0400, Garance A Drosihn wrote:
>=20
> For parsing purposes, I was thinking of something more like:
>=20
>     Revision  Changes    Path
>     1.108     +6  -5     src/sys/kern/kern_descrip.c
>     1.1       +123 0     src/sys/kern/new_file.c    (new)
>     1.234     0 -669     src/sys/kern/dead_file.c   (attic)
>=20
> I'd want the same number of columns under "Changes" for those people
> who parse this by splitting on blanks.

That's a fair point, and the reason I put two columns in in the patch.

How about:

     Revision  Changes    Path
     1.108     +6  -5     src/sys/kern/kern_descrip.c
     1.1       +123 X     src/sys/kern/new_file.c    (new)
     1.234     X -669     src/sys/kern/dead_file.c   (attic)

or some other character to pad the column out, but to indicate that
there's no value there?

Joe

--82I3+IH0IqGh5yIs
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (FreeBSD)
Comment: For info see http://www.gnupg.org

iEYEARECAAYFAjuhcxoACgkQXVIcjOaxUBZn1QCgpcacMct3j5FXdQSInSqHOQ8I
6GwAoOKUW7I1ZNNfzmV9TSU2DKowzcaR
=VZNq
-----END PGP SIGNATURE-----

--82I3+IH0IqGh5yIs--

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message