Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 07 Jul 2007 00:52:44 +0100
From:      Rui Paulo <rpaulo@fnop.net>
To:        "Constantine A. Murenin" <cnst@FreeBSD.org>
Cc:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   Re: PERFORCE change 123027 for review
Message-ID:  <468ED5CC.3030900@fnop.net>
In-Reply-To: <200707061837.l66IbGSN049491@repoman.freebsd.org>
References:  <200707061837.l66IbGSN049491@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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.

-- 
Rui Paulo



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?468ED5CC.3030900>