James Carlson <james.d.carlson at sun.com> writes: > I've (finally!) finished testing of my fix for this issue: > > 440 Need exactly one implementation of scm sniffing. > > This was originally a code review comment from Bill Sommerfeld. The > webrev for this change is here: > > http://cr.opensolaris.org/~carlsonj/webrev-440/ >
which_scm.1:26 'which_scm', not 'webrev' There's a few places (webrev, nightly, ws, xref) which don't explicitly deal with the case where which_scm knows which SCM is in use, but the tool doesn't really know what to do with them. I'd be especially concerned if that in combination with the extra SCMs which_scm will detect lends an illusion of support for those extra SCMs that doesn't in fact exist. I'd also like someone else to take a look through this, rather than just me. -- Rich
