svn commit: r282572 - vendor/OpenBSD/dist/usr.bin/rcs

[Antoine Brodin]

On Thu, May 7, 2015 at 4:42 PM, Bryan Drewery <bdrewery at freebsd.org> wrote:
> On 5/6/2015 6:52 PM, Pedro F. Giffuni wrote:
>> Author: pfg
>> Date: Wed May  6 23:52:15 2015
>> New Revision: 282572
>> URL: https://svnweb.freebsd.org/changeset/base/282572
>>
>> Log:
>>   Bring OpenRCS 20150326 to the vendor area.
>>
>>   A BSD-licensed alternative to GNU Revision Control System.
>>
>>   Obtained from:      OpenBSD
>>
>> Added:
>>   vendor/OpenBSD/dist/usr.bin/rcs/
>>   vendor/OpenBSD/dist/usr.bin/rcs/Makefile   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/buf.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/buf.h   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/ci.1   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/ci.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/co.1   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/co.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/date.y
>>   vendor/OpenBSD/dist/usr.bin/rcs/diff.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/diff.h   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/diff3.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/ident.1   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/ident.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/merge.1   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/merge.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcs.1   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcs.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcs.h   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsclean.1   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsclean.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsdiff.1   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsdiff.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsmerge.1   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsmerge.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsnum.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsparse.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsparse.h   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsprog.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsprog.h   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcstime.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsutil.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rcsutil.h   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rlog.1   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/rlog.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/worklist.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/worklist.h   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/xmalloc.c   (contents, props changed)
>>   vendor/OpenBSD/dist/usr.bin/rcs/xmalloc.h   (contents, props changed)
>>
>
> I'm seeing a lot of discussion about this in private areas. The majority
> of it negative. We might as well get some conversation going.
>
> I have no major opinion on RCS except that it can easily be a package.
> We can provide a package on the release media as well.
>
> However, as long as we have $FreeBSD$ tags in our own source files and
> compiled binaries I find ident(1) to be very useful for debugging. I
> used it just the other day to see what revision someone's /bin/sh was.
> We use it in the ports package builder to get an idea of what revision
> the port is on (it is not 100% accurate since someone may have modified
> distinfo and we are checking Makefile's ident, but it is still very
> useful to give an idea of the range of commits).
>
> Can we please consider at least importing ident(1) and disassociating it
> from WITHOUT_RCS?


Hi,

About WITHOUT_RCS,  freebsd-update(8) uses merge(1) and rc.subr(8)
uses rcs(1), ci(1) and co(1),  so some things no longer work when you
remove rcs.

Cheers,

Antoine