Just a correction to my last post. The commands should have been:
cvs --help-options and
cvs --help update
I just want to say that if you prefer to use the <exec /> command then that is
definitely an option and honestly I would not want to limit that choice. I seem to
remember some complexities with this (using pserver protocol with the System account
comes to mind), but it does work and honestly something might come up that will mean
"just gotta" :-) use a solution like this.
To make a point you could probably use the <exec /> task for compiling your *.cs files
but there are some advantages to using the wrapper. Automatically detecting mcs.exe
over csc.exe (or vbc.exe I think) is one of the advantages you get out of using the
task (1 of the many that are not often thought of I might add). Setting up the
execution environment and allowing easy override of system variables is another. The
<cvs /> task provides the latter and the former is something I am working towards.
You may be right, maybe the <globaloptions /> is a bad idea and should go the way of
the dodo :-). Maybe they are something that should be abstracted out into a generic
"commandoptions" set. My point is there will need to be some work done to ensure that
options don't clash (i.e. -D for a global means something different than -D for a
command option), and I am sure other things will pop up :-). This is work that I will
gladly do but I would like to get some feedback on how important this is before I
change it. I don't want to spend a lot of my free time on this if it is not an issue.
Cheers,
Clayton
> -----Original Message-----
> From: Jaroslaw Kowalski [mailto:[EMAIL PROTECTED]
> Sent: March 14, 2004 11:38 AM
> To: Clayton Harbour; [EMAIL PROTECTED]
> Subject: Re: [Nant-users] CVS update
>
>
> 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_id70&alloc_id638&op=click
_______________________________________________
Nant-users mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/nant-users
