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_id70&alloc_id638&op=click
_______________________________________________
Nant-users mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-users
