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

Reply via email to