Re: defadvice question.

[Michal]

hallo Adreas

Andreas Politz <politza@fh-trier.de> writes:

> I suspect that it does work, just not the way you expect it :
> `add-to-list' does not insert duplicate elements into a list. That
> means, if CVSROOT is already at some position in the
> `minibuffer-history', this advice does nothing.

this is not on the list yet.

>
> On the other hand, is this even necessary ? Peeking at the
> `cvs-get-cvsroot' suggests, that CVSROOT is the default value anyway
> (when calling `cvs-checkout' w/o prefix), unless you are already in a
> cvs directory or the `cvs-root' variable is set.

I am calling cvs-checkout with prefix, because I need to add some
parameters to cvs ( -r BRANCH_NAME), so it also asks about the cvs root.

best regards,
MIchal