Date: Thu, 31 Jan 2002 16:52:52 +0100 From: Eivind Eklund <eivind@FreeBSD.ORG> To: Ruslan Ermilov <ru@FreeBSD.ORG> Cc: Sheldon Hearn <sheldonh@starjuice.net>, arch@FreeBSD.ORG Subject: Re: Adding support for a global src tree serial number Message-ID: <20020131165252.A96747@phoenix.dmnshq.net> In-Reply-To: <20020131144345.A73522@sunbay.com>; from ru@FreeBSD.ORG on Thu, Jan 31, 2002 at 02:43:45PM %2B0200 References: <20020131140403.A69232@sunbay.com> <79636.1012479657@axl.seasidesoftware.co.za> <20020131144345.A73522@sunbay.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Jan 31, 2002 at 02:43:45PM +0200, Ruslan Ermilov wrote: > On Thu, Jan 31, 2002 at 02:20:57PM +0200, Sheldon Hearn wrote: > > On Thu, 31 Jan 2002 14:04:03 +0200, Ruslan Ermilov wrote: > > > This scheme won't work because the state of the tree can be modified > > > by CVS meisters performing direct operations on a repository. See > > > how stealthy the latest GCC import gone. > > > > I don't think there are so many CVS meisters, or so much src repo > > surgery, that having them bump the serial number manually in these cases > > is a problem. > > "commit" is atomic operation, while direct manipulation of repository > isn't. To make it atomic, CVS meisters would have to lock src/, make > the necessary surgery, bump serial number, then unlock it. 'cvs commit' is non-atomic. We will not be able to get this to work perfectly with cvs and cvsup; both are non-atomic. We could make cvs atomic for our use, I guess, but I suspect we'd find that non-workable (as it would mean that we would get a lot more latency for any operations on the cvs tree). I still think a /usr/src/SERIAL would be nice; thought it would not give perfect information, it would still give better information than not having it. Eivind. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020131165252.A96747>