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

Reply via email to