Better handle stripping CVSROOTs from CVS paths.

Review Request #1684 — Created June 26, 2010 and submitted

Information

Review Board

Reviewers

Better handle stripping CVSROOTs from CVS paths.

We had code to strip the absolute path prefixes from CVS paths in diffs,
but it didn't handle CVSROOTs that didn't separate the domain and path
with a ':'. We now handle this case a bit better by finding where that
path begins.

Also fixes a bug where the attic filename was being generated incorrectly
if the file didn't contain a '/' in the name.
Tested with a diff I was working with locally. I no longer had errors when viewing it with my repository.
david
  1. Looks good. Should this have any bugs marked?
    1. Not that I've seen. I think everyone pretty much uses the ":" delimiter, while we don't appear to include that when we build the path. So it's triggered by our new CVS stuff on SourceForge, basically, which I imagine isn't strongly used.
  2. 
      
Loading...