From owner-p4-projects@FreeBSD.ORG Sat Jul 7 00:37:25 2007 Return-Path: X-Original-To: p4-projects@freebsd.org Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id F0A0116A46D; Sat, 7 Jul 2007 00:37:24 +0000 (UTC) X-Original-To: perforce@FreeBSD.org Delivered-To: perforce@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id ACCF816A400; Sat, 7 Jul 2007 00:37:24 +0000 (UTC) (envelope-from cnst@FreeBSD.org) Received: from mojo.ru (mojo.ru [84.252.152.63]) by mx1.freebsd.org (Postfix) with ESMTP id 3276413C44C; Sat, 7 Jul 2007 00:37:23 +0000 (UTC) (envelope-from cnst@FreeBSD.org) Received: from [192.168.0.16] (nc-76-4-28-21.dhcp.embarqhsd.net [76.4.28.21]) (authenticated bits=0) by mojo.ru (8.12.11.20060308/8.12.10) with ESMTP id l670LgO7022293 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sat, 7 Jul 2007 04:21:45 +0400 Message-ID: <468EDC8F.9080909@FreeBSD.org> Date: Fri, 06 Jul 2007 20:21:35 -0400 From: "Constantine A. Murenin" User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.7.5) Gecko/20041217 X-Accept-Language: en-gb, en-gb-oed, en, en-us, ru, ru-ru, ru-su MIME-Version: 1.0 To: Rui Paulo References: <200707061837.l66IbGSN049491@repoman.freebsd.org> <468ED5CC.3030900@fnop.net> In-Reply-To: <468ED5CC.3030900@fnop.net> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Cc: Shteryana Shopova , Perforce Change Reviews , "Constantine A. Murenin" Subject: Re: PERFORCE change 123027 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 07 Jul 2007 00:37:25 -0000 On 06/07/2007 19:52, Rui Paulo wrote: > Constantine A. Murenin wrote: > >>http://perforce.freebsd.org/chv.cgi?CH=123027 >> >>Change 123027 by cnst@dale on 2007/07/06 18:37:09 >> >> Improved the script, now it creates the dev/lm directory if it doesn't exist, and renames the files if they already exist, so that the new files can be linked. (Previously, it was just giving errors in these circumstances.) >> >>Affected files ... >> >>.. //depot/projects/soc2007/cnst-sensors/sensors.ln.sh#2 edit > > > I think that using a script to do this is wrong. > Is it possible that you could create a branch for your project based on > the freebsd vendor branch ? Then you could integrate it from time to > time. I know you don't like p4, but this is very easy to do. > This way, you can create a diff against HEAD more easily and it also > allows other folks to test your patches/drivers without any hassle. > > If you need any help, just don't hesitate to ask me. Thank you for your concern, rpaulo! All files that exist in FreeBSD are already branched in my own branch from the freebsd vendor branch, hence I will in fact do integration from time to time as you suggest. I think I will use the current scheme for the moment -- branching only the files/directories that I am directly going to work on. I will then see if any other kinds of branching will be necessary, and will act accordingly. Also note that my script is not destructive -- it doesn't remove any files, except for links. ;) Cheers, Constantine.