Date: Mon, 14 Jan 2002 20:57:02 +0200 From: Giorgos Keramidas <keramida@ceid.upatras.gr> To: Ruslan Ermilov <ru@FreeBSD.ORG> Cc: audit@FreeBSD.ORG Subject: Re: last with -y option Message-ID: <20020114185702.GA30454@hades.hell.gr> In-Reply-To: <20020114172454.G21333@sunbay.com> References: <20020111232631.GA66629@hades.hell.gr> <20020114172454.G21333@sunbay.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2002-01-14 17:24:54, Ruslan Ermilov wrote:
> On Sat, Jan 12, 2002 at 01:26:31AM +0200, Giorgos Keramidas wrote:
> > The patch that was sent in PR bin/12982 does not apply to the
> > usr.bin/last (probably because our last was modified for WARNS=2 after
> > the patch was submitted), but I liked the idea that it describes.
> >
> > The attached patch adds a -y option to last that will print the year
> > too when listing wtmp records. It also removes a few EOL whitespaces
> > that I found while reading the last.c source.
> >
> > If this does not break any standards, and you don't think that it
> > breaks last(1) in some way, I'll have to find myself an src/ committer
> > to take care of this PR :-)
> >
> There're two functional and style bugs in your patch, see below.
iewdose pointed out that many parts of the last.c source are
duplicated in a lot of other places. I'll try to come up with changes
that remove these duplicates first, and then add the -y option.
This patch will be obsolete by then, but thanks Ruslan for taking the
time to read/comment the changes.
> Please don't commit unrelated whitespace changes together with functional
> ones, commit them seperately.
Good point. Thanks :)
I'm not committing to src/ (apart from manpages) without approval anyway,
but you're right. If/when last.c is changed I'll keep in mind to
separate whitespace and content changes.
--
Giorgos Keramidas . . . . . . . . . keramida@{ceid.upatras.gr,freebsd.org}
FreeBSD Documentation Project . . . http://www.freebsd.org/docproj/
FreeBSD: The power to serve . . . . http://www.freebsd.org/
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-audit" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020114185702.GA30454>