[Buildbot-devel] Adding -N to cvs co line to allow cvs alias's which specify multiple modules to be checked out

[Bill Deegan]

Greetings,

I'm working on a patch which will allow the user to specify "-N" to
cvs co command which buildbot executes.
Should I make this generic?
For example there's global_options you can currently specify which is
options run after cvs but before any
cvs verb.
I was creating a checkout_options argument to the buildstep which
would put arbitrary arguments before any -r or -D arguments to cvs
checkout only.

Or should I just add a flag to instruct buildbot to add -N to the checkout line.

For those not familiar the cvs manpage specifies -N's functionality as follows:
       -N

         Only useful together with -d dir.  With this option, cvs will
not ``shorten''  module  paths  in
         your  working  directory when you check out a single module.
See the -d flag for examples and a
         discussion.

I need this for a project where the developers use cvs co build, which
checks out directories src,tests, and other
without -N the checkout fails because cvs will remove the src,tests,
and other directory names.

Thanks for any advice.
-Bill