I know that this is the requirement of the cvs client. I just think that <cvs-*> tasks should be smart and isolate the user from the complexities of the cvs arguments. If they aren't I think I'm better off with:
<exec program="cvs" commandline="-Pd ..." /> which (modulo authentication) works like a charm. Provided that cvs is similar to svn, once we have the <cvs-*> tasks which are command-line free we could provide exactly the same interface for <svn-*> tasks. What do you think? Jarek ----- Original Message ----- From: "Clayton Harbour" <[EMAIL PROTECTED]> To: "Jaroslaw Kowalski" <[EMAIL PROTECTED]>; <[EMAIL PROTECTED]> Sent: Sunday, March 14, 2004 8:23 PM Subject: RE: [Nant-users] CVS update > Jarek, > > Splitting the option sets into a global option set and a command specific option set is something done in cvs already, not something I am introducing. If you are unsure of what I mean please type: > cvs --help-options > and > cvs --help-options update > > This also provides documentation on which command option is global and which is command specific. > > For reference the ant <cvs /> task handles this by placing an order dependency on the command line arguments (http://ant.apache.org/manual/CoreTasks/cvs.html), and moving the command itself (i.e. checkout, diff, update) into the option set. I don't like this because I think it means you have to understand more about cvs than the way I propose (i.e. you will have to know when to specify the cvs root, the module, etc.) but if there is more feedback that this is what people want then it is do-able. > > I appreciate the input and I would like to get some more feedback from everyone who might be concerned with this. As it stands I have really only seen your ideas and mine. Really :-). > > > Thanks, > > Clayton > > > > > -----Original Message----- > > From: Jaroslaw Kowalski [mailto:[EMAIL PROTECTED] > > Sent: March 14, 2004 10:01 AM > > To: Clayton Harbour; [EMAIL PROTECTED] > > Subject: Re: [Nant-users] CVS update > > > > > > Why do you want to split options into "global" and "command" > > ? This introduces a new class of problem: "which option is > > global and which is command". > > > > Users don't like it. Really. > > > > Jarek > > ----- Original Message ----- > > From: "Clayton Harbour" <[EMAIL PROTECTED]> > > To: "Jaroslaw Kowalski" <[EMAIL PROTECTED]>; "Narendra Kulkarni" > > <[EMAIL PROTECTED]>; <[EMAIL PROTECTED]> > > Sent: Sunday, March 14, 2004 6:37 PM > > Subject: RE: [Nant-users] CVS update > > > > > > > Hi Jarek, > > > > > > I don't really like the idea of specifying options as > > attributes, but > > > I > > think there is some common ground here: > > > <cvs-update globaloptions="-nt" > > > commandoptions="-r VERSION_FOO" ...... > > > > <globaloptions> > > > <option name="readonly" value="true"/> > > > </globaloptions> > > > <commandoptions> > > > <option name="pergeempty" value="false"/> > > > <option name="newdirs" value="false"/> > > > </commandoptions> > > > <fileset> > > > <includes name="**/*.cs"/> > > > </fileset> > > > </cvs-update> > > > > > > where: > > > n = no modifications done on local file system > > > t = show trace of program execution > > > > > > If someone is concerned about task size they can just use the > > > commandline > > options, however if they prefer to be verbose (unfortunately > > my preference > > (-:) then the option is there. This also makes a fairly > > generic interface that I can reuse for other version control > > systems (i.e. the <svn /> task in NAntContrib). How does that sound? > > > > > > > > > Clayton > > > > > > > > > > -----Original Message----- > > > > From: Jaroslaw Kowalski [mailto:[EMAIL PROTECTED] > > > > Sent: March 14, 2004 8:54 AM > > > > To: Clayton Harbour; Narendra Kulkarni; > > > > [EMAIL PROTECTED] > > > > Subject: Re: [Nant-users] CVS update > > > > > > > > > > > > I don't think it's a problem. > > > > > > > > Users don't usually think in terms of command line options (maybe > > > > except "cvs update -Pd" and "cvs diff -u" because they are so > > > > common) so this would lead to an increased readability. > > > > > > > > I suggest the following syntax (put all options in a single > > > > attribute): > > > > > > > > <cvs-update purgeempty="true" newdirs="false" > > overwritelocal="true" > > > > extraoptions="-D ..." /> > > > > > > > > What do you think? > > > > > > > > Jarek > > > > > > > > ----- Original Message ----- > > > > From: "Clayton Harbour" <[EMAIL PROTECTED]> > > > > To: "Jaroslaw Kowalski" <[EMAIL PROTECTED]>; "Narendra Kulkarni" > > > > <[EMAIL PROTECTED]>; <[EMAIL PROTECTED]> > > > > Sent: Sunday, March 14, 2004 5:32 PM > > > > Subject: RE: [Nant-users] CVS update > > > > > > > > > > > > > Hi, > > > > > > > > > > More meaningful names does make a lot of sense (I have this > > > > in place > > > > > for sticky-tag and override-directory). I would prefer to > > > > stay with > > > > > the option collection simply because using attributes for > > > > every option > > > > > available will get messy. So maybe something like: > > > > > > > > > > <cvs-update ...... > > > > > > <commandoptions> > > > > > <option name="pergeempty" value="false"/> > > > > > <option name="newdirs" value="false"/> > > > > > </commandoptions> > > > > > </cvs-update> > > > > > > > > > > Where the -P, -d options would also be accepted. I > > also think the > > > > > default on switch makes sense for these, which is where > > the value > > > > > "true" and "false" would be used to shut these (or any > > > > other option) > > > > > off. A blank or missing value would default to "true" > > to preserve > > > > > backwards compatibility. > > > > > > > > > > If this sounds good I will start implementing this. > > > > > > > > > > > > > > > Cheers, > > > > > > > > > > Clayton > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > From: Jaroslaw Kowalski [mailto:[EMAIL PROTECTED] > > > > > > Sent: March 14, 2004 2:14 AM > > > > > > To: Clayton Harbour; Narendra Kulkarni; > > > > > > [EMAIL PROTECTED] > > > > > > Subject: Re: [Nant-users] CVS update > > > > > > > > > > > > > > > > > > Clayton, > > > > > > > > > > > > Wouldn't it be better to implement some meaningful attributes > > > > > > for this? > > > > > > > > > > > > <cvs-update newdirs="true" purgeempty="true" /> > > > > > > > > > > > > (BTW. I believe -Pd are so common that they should be on > > > > by default > > > > > > and there should be an option to turn them off for > > > > special ocasions) > > > > > > > > > > > > Jarek > > > > > > > > > > > > ----- Original Message ----- > > > > > > From: "Clayton Harbour" <[EMAIL PROTECTED]> > > > > > > To: "Narendra Kulkarni" <[EMAIL PROTECTED]>; > > > > > > <[EMAIL PROTECTED]> > > > > > > Sent: Sunday, March 14, 2004 6:31 AM > > > > > > Subject: RE: [Nant-users] CVS update > > > > > > > > > > > > > > > > > > Hi Narendra, > > > > > > > > > > > > Using the latest nightly builds you will have to do 2 things: > > > > > > 1) set the usesharpcvslib attribute to false (either > > > > the task > > > > > > level attribute or there is a global/ project level attribute) > > > > > > 2) send in the command options -Pd; please > > double check the > > > > > > actual syntax but it should look something like this: > > > > > > <cvs-update command="update" ...... > > > > > > > <commandoptions> > > > > > > <option name="-P" value=""/> > > > > > > <option name="-d" value=""/> > > > > > > </commandoptions> > > > > > > </cvs-update> > > > > > > > > > > > > Just a note, if you are running as a service and using > > > > pserver you > > > > > > will also need to specify a password file using the passfile > > > > > > attribute. This attribute takes the full path (including > > > > filename) > > > > > > to your .cvspass file as an argument. > > > > > > > > > > > > > > > > > > Cheers, > > > > > > > > > > > > Clayton > > > > > > > > > > > > > > > > > > PS: Thanks for the reminder :-) > > > > > > > > > > > > -----Original Message----- > > > > > > From: Narendra Kulkarni [mailto:[EMAIL PROTECTED] > > > > > > Sent: March 12, 2004 2:12 PM > > > > > > To: [EMAIL PROTECTED] > > > > > > Subject: [Nant-users] CVS update > > > > > > > > > > > > > > > > > > Hi, > > > > > > I noticed that CVS-UPDATE does not create missing > > > > directories. Is > > > > > > there any way to do this? thanks. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Narendra Kulkarni > > > > > > > > > > > > Software Engineer II > > > > > > Veridyne, inc. > > > > > > Your technology partner. Worldwide. > > > > > > > > > > > > > > > > > > > > > > > > Email: > > > > > > [EMAIL PROTECTED] > > > > > > > > > > > > Office: > > > > > > (703) 412 - 5072 (ext 238) > > > > > > > > > > > > Fax: > > > > > > (703) 412 - 5078 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------------------------------- > > > > > This SF.Net email is sponsored by: IBM Linux Tutorials > > Free Linux > > > > > tutorial presented by Daniel Robbins, President > > > > and CEO of > > > > > GenToo technologies. Learn everything from fundamentals > > to system > > > > > administration.http://ads.osdn.com/?ad_id70&alloc_id638&opĚk > > > > > _______________________________________________ > > > > > Nant-users mailing list > > > > > [EMAIL PROTECTED] > > > > > https://lists.sourceforge.net/lists/listinfo/nant-users > > > > > > > > > > > > > > > > > > > > > > ------------------------------------------------------- > > > This SF.Net email is sponsored by: IBM Linux Tutorials > > > Free Linux tutorial presented by Daniel Robbins, President > > and CEO of > > > GenToo technologies. Learn everything from fundamentals to system > > > administration.http://ads.osdn.com/?ad_id70&alloc_id638&opĚk > > > _______________________________________________ > > > Nant-users mailing list > > > [EMAIL PROTECTED] > > > https://lists.sourceforge.net/lists/listinfo/nant-users > > > > > > > > ------------------------------------------------------- This SF.Net email is sponsored by: IBM Linux Tutorials Free Linux tutorial presented by Daniel Robbins, President and CEO of GenToo technologies. Learn everything from fundamentals to system administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click _______________________________________________ Nant-users mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/nant-users
