Dietmar Petras writes:
> 
> alloc_pending() tests pending_error and returns TRUE if (pending_error!=0).
> Thus pending_error_text is never set.

Thanks!  People have been reporting that useless error message for quite
a while -- I assumed (silly me) that it was comming from one of the
places that set pending_error without setting pending_error_text, so
I've been adding a bunch more messages; unfortunately, I've been copying
the defective code.  :-(  I've now corrected it all along the lines you
suggested and checked it in.

> > cvs status Kernel/Makefile 
> cannot create_adm_p /tmp/cvs-serv9476/Kernel
> Keine Berechtigung (german for: permission denied)
> 
> After going into the deeps of create_adm_p() I found that cvs did not find
> CVSROOT/Emptydir and was not allowed to create it.
[...]
> Wouldn't it be a good idea to check for the existence of CVSROOT/Emptydir
> and to report its nonexistence to the user and ask him to run cvs init as
> root?

I don't see why Emptydir should be treated any differently than any
other required part of the repository -- if you're going to go around
deleting stuff without knowing what you're doing, you deserve a cryptic
error message.  ;-)

-Larry Jones

I think my cerebellum just fused. -- Calvin

Reply via email to