From owner-svn-src-vendor@FreeBSD.ORG Thu May 7 14:58:49 2015 Return-Path: Delivered-To: svn-src-vendor@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id AA0EAE06; Thu, 7 May 2015 14:58:49 +0000 (UTC) Received: from mail-wi0-x236.google.com (mail-wi0-x236.google.com [IPv6:2a00:1450:400c:c05::236]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 5777010E8; Thu, 7 May 2015 14:58:49 +0000 (UTC) Received: by wizk4 with SMTP id k4so246273635wiz.1; Thu, 07 May 2015 07:58:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:cc:content-type; bh=YodtKKdaL9FenwCfyYjs91lH14BeMMFRhSibUUkxjvc=; b=V0oZVxT2ukHfMMsEzgU8W2JToHP/KXWfllyrZgGGNYdKRlTo2/A5e1otlCfVkb1vNc ApoyZFbPqXrSyJ9upWfYcNjIELfiKaHgH6q7+oAOADGeGiUeWjX3BoGesyxqj9bsFGT7 KH15agnRrMKBaQ0+N4QwCc7GJDCF/AZ//HK5asNnzsg48UiHnR21QWXippIFrhPHEnNi vMReVw3jsrKq+Fc2mMsQ9XSdpTTXG2+nFseDnH+jSpnMAhH05gMKUHutqGBb5k53rM0K CFBkrcHWoB2OFceJeUawx4w1CN9iQ+5o6yqxnzozOY/Ld59oNOvFdL5mBTraaN2yHPFP FvPQ== MIME-Version: 1.0 X-Received: by 10.194.2.16 with SMTP id 16mr8323672wjq.46.1431010727765; Thu, 07 May 2015 07:58:47 -0700 (PDT) Sender: antoine.brodin.freebsd@gmail.com Received: by 10.194.17.136 with HTTP; Thu, 7 May 2015 07:58:47 -0700 (PDT) In-Reply-To: <554B79F2.2010103@FreeBSD.org> References: <201505062352.t46NqGRq020002@svn.freebsd.org> <554B79F2.2010103@FreeBSD.org> Date: Thu, 7 May 2015 16:58:47 +0200 X-Google-Sender-Auth: 4v-k0QYM8IT_GtMbK3xt2-xGatY Message-ID: Subject: Re: svn commit: r282572 - vendor/OpenBSD/dist/usr.bin/rcs From: Antoine Brodin To: Bryan Drewery Cc: "Pedro F. Giffuni" , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-vendor@freebsd.org Content-Type: text/plain; charset=UTF-8 X-BeenThere: svn-src-vendor@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the vendor work area tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 07 May 2015 14:58:49 -0000 On Thu, May 7, 2015 at 4:42 PM, Bryan Drewery 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