CVS scmtool

Review Request #50 — Created June 4, 2007 and submitted — Latest diff uploaded

Information

Review Board SVN (deprecated)
trunk

Reviewers

Update:
Added 100% unittests
Removed extrapath dependency

====
Update:
Back from vacation, fixed the remaining code format issues.
Interface now lists new files with correct name
Open problem: sort out and remove dependency of extrapath and add tests.

====
Update:
Uses the new diffparser, code gets a lot better now.
Open problem: interface lists new files as "/dev/null" (file.origName), not the real name found in file.newName, is this changes recently?
Also, removes dependency on external lib for CVS, needs to think a smart way to use other methods than pserver.

====
A first draft of the CVS scmtool. Whats needs to be done is to figure out how to store and display the complete filename when viewing the diff and change to a better cvs lib.
100% unit-tested (included)

Diff Revision 8 (Latest)

orig
1
2
3
4
5
6
7
8
trunk/reviewboard/scmtools/cvs.py
trunk/reviewboard/scmtools/tests.py
trunk/reviewboard/scmtools/fixtures/initial_data.json
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/checkoutlist
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/checkoutlist,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/commitinfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/commitinfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/config
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/config,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/cvswrappers
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/cvswrappers,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/editinfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/editinfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/history
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/loginfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/loginfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/modules
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/modules,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/notify
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/notify,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/rcsinfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/rcsinfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/taginfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/taginfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/verifymsg
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/verifymsg,v
trunk/reviewboard/scmtools/testdata/cvs_repo/test/testfile,v
trunk/reviewboard/scmtools/cvs.py
trunk/reviewboard/scmtools/tests.py
trunk/reviewboard/scmtools/fixtures/initial_data.json
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/checkoutlist
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/checkoutlist,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/commitinfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/commitinfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/config
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/config,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/cvswrappers
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/cvswrappers,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/editinfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/editinfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/history
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/loginfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/loginfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/modules
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/modules,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/notify
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/notify,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/rcsinfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/rcsinfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/taginfo
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/taginfo,v
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/verifymsg
trunk/reviewboard/scmtools/testdata/cvs_repo/CVSROOT/verifymsg,v
trunk/reviewboard/scmtools/testdata/cvs_repo/test/testfile,v
Loading...