Re: warning when using checkout command

Milos Kleint wrote:
> 
> Martin Entlicher wrote:
> 
> >Milos Kleint wrote:
> >
> >>however the condition there is  not 100% bulletproof.
> >>it won't work for checkout, export, rlog, rannotate and maybe import.
> >>(and other commands that don't use the local directory structure)
> >>the catch is that these commands have a module-expansion request. which
> >>acts as another step in the communication.
> >>because your module can be an alias module (defined in the
> >>CVSROOT/modules file I think) which actually describes multiple real
> >>modules  (real directories in the cvs server root directory), the
> >>module-expand command is used to translate these symbolic modules to the
> >>real thing.
> >>then the actual command request is sent with the proper modules (already
> >>translated/converted)
> >>
> >>
> >
> >I see. So either the library is mistakenly reusing the connection itself
> >(CheckoutCommand, etc.), or the condition should be different.
> >
> >
> 
> no the use cases are correct, the condition should be different. or at
> least the words in the warning..
> 

I've filed issue #33920 for this problem. It's scheduled to be fixed for 4.0.

-Martin