Add a consistent function in the SVN backends for getting history logs.
Review Request #5978 — Created June 11, 2014 and submitted
PySVN and Subvertpy had very different ways of getting history logs from
a repository. Both had functions that accepted paths and revision
ranges, but they had different defaults for these parameters and
represented the results in different ways.This adds a
get_log()function for the SVN client backends that return
entries in consistent ways. It'll soon be used to centralize some of the
logic that is currently duplicated between the backends.
Made use of these in an upcoming change. This particular change isn't unit tested,
but the changes making use of this data are.
| Description | From | Last Updated |
|---|---|---|
|
strict_node_history isn't a very self-explanatory term. Maybe something like ignore_before_branch? |
 david
|
-
This is a review from Review Bot. Tool: Pyflakes Processed Files: reviewboard/scmtools/svn/subvertpy.py reviewboard/scmtools/svn/pysvn.py reviewboard/scmtools/svn/base.py Ignored Files:
-
-
reviewboard/scmtools/svn/base.py (Diff revision 1) strict_node_historyisn't a very self-explanatory term. Maybe something likeignore_before_branch?
Change Summary:
- Normalized paths in
get_log(). - Reanmed
strict_node_historytolimit_to_path.
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 2 (+105 -1) |
-
This is a review from Review Bot. Tool: PEP8 Style Checker Processed Files: reviewboard/scmtools/svn/subvertpy.py reviewboard/scmtools/svn/pysvn.py reviewboard/scmtools/svn/base.py Ignored Files:
This is a review from Review Bot. Tool: PEP8 Style Checker Processed Files: reviewboard/scmtools/svn/subvertpy.py reviewboard/scmtools/svn/pysvn.py reviewboard/scmtools/svn/base.py Ignored Files: