[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Export files by specific revision number
From: |
Spiro Trikaliotis |
Subject: |
Re: Export files by specific revision number |
Date: |
Sat, 24 Sep 2005 11:56:35 +0200 |
User-agent: |
Mutt/1.5.9i |
Hello again,
* On Fri, Sep 23, 2005 at 01:45:25PM -0400 Larry Jones wrote:
> Spiro Trikaliotis writes:
> >
> > So, if I export a whole module (or anything more than just one file), a
> > symbolic tag has to be given. If I export only one single file, a
> > revision would be allowed. Would that be an acceptable solution?
>
> Sounds good to me.
Ok, Now, the question is: How to implement the cleaneast way to find out
if more than one file is addressed?
After looking through the code, I see two possibilities:
1. In get_server_responses() (client.c), I could check if I get more
than one "Created".
Unfortunately, this means I have to test much more cases as this
function is used by the other cvs commands, too. Furthermore, when I
find out there was more than one file, I already have created some
others.
2. When performing an cvs export, do the equivalent of "cvs -nq co" to
find out if there is more than one file beforehand.
I see another, 3rd possibility (which is much easier):
3. Just remove that
if (tag && isdigit (tag[0]))
error (1, 0, "tag `%s' must be a symbolic tag", tag);
test and let the user decide what he wants. That is, if he wants to
get rev x.y of the whole repository, let him do it (a warning might
be appropriate, but why should we prevent him from doing so?)
Of course, no. 3 is the easiest solution. In fact, it is the one I would
prefer :), and this is not only because I'm lazy.
> > This is not the fix I had in mind. ;)
>
> I know, but I felt it was worth doing anyway. Just be sure to change
> it again in your patch. :-)
Well, the documentation would have liked the same patch, too.
> > I am never prompted for a password. Anyway, for other servers, this
> > works perfectly.
>
> I'm not prompted for a password either, but it appears that
> keyboard-interactive is the method that succeeds:
[...]
> Perhaps you need a newer version of ssh?
I tried with Cygwin:
OpenSSH_4.2p1, OpenSSL 0.9.8 05 Jul 2005
and Debian/Sarge:
OpenSSH_3.8.1p1 Debian-8.sarge.4, OpenSSL 0.9.7e 25 Oct 2004
both failed.
Regards,
Spiro.
--
Spiro R. Trikaliotis http://cbm4win.sf.net/
http://www.trikaliotis.net/ http://www.viceteam.org/