Re: [BUG RFC/PATCH] git-cvsimport

From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Aug 16, 2010, 17:43 UTC
Message-ID: <AANLkTimXAZMfXWjcb7RKE2_y6_v7MHQRK7oxK42n-5C0@mail.gmail.com>
In-Reply-To: <7vpqxicyoy.fsf@alter.siamese.dyndns.org>

On Mon, Aug 16, 2010 at 16:25, Junio C Hamano <gitster@pobox.com> wrote:

Show 15 quoted lines

> Michael Haggerty <mhagger@alum.mit.edu> writes:
>
>> My guess is that cvsps is using CVS commands to access the test CVS
>> repository, and that CVS wants to write to the file CVSROOT/history to
>> log what is being done.  The logging behavior can be turned off either:
>> ...
>> Apparently -R and "$CVSREADONLYFS" were added in CVS 1.12.1, which was
>> released in 2003.
>
> That unfortunately does not solve much.  t9602 seems to want to create a
> new CVSROOT/val-tags file, so if Ævar wants to keep the source tree
> read-only, the test CVS repository needs to be copied to a scratch place.
>
> Perhaps something like this?  I actually have to wonder why cvsIMPORT
> needs to write into anywhere, though.

That looks good. It's pretty much exactly what I was going to submit, but I was waiting until you applied my other patches to pu (since they'd with that patch as I changed the setup code in the skip patches too).