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