Date: Sun, 20 May 2001 14:51:52 +0300 (EEST) From: Maxim Sobolev <sobomax@mail-in.net> To: ru@FreeBSD.ORG Cc: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/usr.bin Makefile src/usr.bin/scrshot Makefile scrshot.1 scrshot.c src/usr.sbin/vidcontrol vidcontrol.1 vidcontrol.c Message-ID: <200105201151.f4KBppr02728@mail.uic-in.net> In-Reply-To: <20010520141504.E3338@sunbay.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 20 May 2001 14:15:04 +0300, Ruslan Ermilov wrote: > On Fri, May 18, 2001 at 11:47:36PM -0700, Maxim Sobolev wrote: > > sobomax 2001/05/18 23:47:36 PDT > > > > Modified files: > > usr.bin Makefile > > usr.sbin/vidcontrol vidcontrol.1 vidcontrol.c > > Removed files: > > usr.bin/scrshot Makefile scrshot.1 scrshot.c > > Log: > > Move scrshot(1) functionality into vidcontrol(1). > > > > Suggested by: many > > Not objected to by: nik (scrshot co-author) > > > > Revision Changes Path > > 1.160 +1 -2 src/usr.bin/Makefile > > 1.35 +138 -1 src/usr.sbin/vidcontrol/vidcontrol.1 > > 1.37 +86 -4 src/usr.sbin/vidcontrol/vidcontrol.c > > > You should have waited longer in a review phase... Yes, I know that normal review period should be much longer than that, but this case was slightly special because of the following reasons: 1) It was necessary to minimise effect of introduction and subsequent removal of scrshot(1) utility (read number of -current users that will get stale /usr/bin/scrshot and corresponding manpage). 2) One of the primary authors of the tool (nik) told that he has absolutely no problems with that change. Several other committers agreed that scrshot functionality has to be merged into vidcontrol. 3) The change was pretty trivial. In any case I'm apologising if that caused any problems to you or to anybody else. -Maxim To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200105201151.f4KBppr02728>