On Wed 06 Sep 2006 at 10:55AM, Mike Kupfer wrote:
> There's a version of webrev with Subversion support in the (internal)
> SVN repo at file:///net/tonic-gate.sfbay.sun.com/gates/scm-host-repo
> (under cmd/webrev/).
> 
> If you don't want to pull the entire repository just to look at
> webrev.sh, there's a copy at /home/kupfer/tonic/scm-repo/.

Cool, I took a look.  I have a couple of questions:

        - Would it be possible to autodetect that one is using SVN?
          One of the changes I am making to webrev is to try to
          have it "just do the right thing" if you just invoke it
          as 'webrev'-- my hope is that it could be a tool of interested
          to broader users of SVN, Hg, etc. if it "just worked"

        - Would it make sense to have a 'mode' flag instead of say,
          -S?  For example, we could -m tw, -m svn, -m hg as an
          extensible way of supporting multiple SCM types.  (One of my
          changes is to have webrev use getopts(1))

Otherwise, it looks like with a little effort this will mostly "drop
in" to what I have.  Thanks,

        -dp

-- 
Daniel Price - Solaris Kernel Engineering - [EMAIL PROTECTED] - blogs.sun.com/dp
_______________________________________________
tools-discuss mailing list
[email protected]

Reply via email to