Age | Commit message (Collapse) | Author |
|
This adds improved logic for determining the parent branch used when generating
a diff. It will attempt to detect the merge base of the current head, falling
back to defaults if necessary. A tracking branch can be manually specified
too.
The new code attempts the following:
1) Tries to detect the tracking branch for the current HEAD, if possible.
2) If a tracking branch doesn't exist, or is not remote, it will fall back
on 'origin/master'.
3) If the --tracking-branch parameter is specified, this branch will be used
instead of 'origin/master'.
It also provides unit tests for post-review. Right now, these test only
GitClient (in pure Git mode), but new tests can be added down the line for
the rest of post-review.
Patch by Dan Savilonis
Reviewed at http://reviews.reviewboard.org/r/1144/
|
|
If a .reviewboardrc file had a syntax error, we would silently ignore it,
which made it hard to debug problems. We now fail on syntax errors and
list the error information, including the filename, line number and offset on
the line.
Patch by Eric Huss
|
|
The fix for bug #585 broke the binary file support. This change brings it
back.
This also fixes the Perforce server versioning check.
Patch by Eric Huss
Reviewed at http://reviews.review-board.org/r/1154/
|
|
Some people have source files that don't end in a newline. In this case, it's
possible for us to create a diff file where the end of one file is joined on the
same line as the header for the next file. This change adds an extra check to
_do_diff to make sure that we don't make broken diffs when people have broken
files.
Fixes bug 1340.
|
|
When creating an SVN diff for a remote repository using --revision-range and
passing in specific filenames, post-review would only get the first file as the
diff. This is because it was using the arg as the base path. This change makes
it treat the arg as a file list if it's more than one item long.
Fixes bug 1162
|
|
Patch from Flavio Castelli. With this patch post-review will support the
revision-range option following the syntax explained at
http://www.review-board.org/docs/manual/dev/users/tools/post-review/#posting-committed-code
Testing done:
I have tested this new feature locally and it works fine.
I have successfully tested the following commands:
post-review --revision-range=rev1:rev2 --guess-summary --guess-description
post-review --revision-range=rev1:rev2 --guess-description
post-review --revision-range=rev1:rev2
post-review --revision-range=rev --guess-summary --guess-description
post-review --revision-range=rev
Reviewed at http://reviews.review-board.org/r/1049/
|
|
When setting a field (such as target reviewers) from post-review, we were
saving the draft, always. This is bad enough, but we were doing this before
even updating the diff. This would result is a published draft and an extra
change description entry. We no longer save the draft, unless the user
uses --publish.
Fixes bug #811
|
|
We were hard-coding the path to the user's Application Data directory, which
changed in Windows Vista. We now use the APPDATA environment variable, which
is guaranteed to point to the correct directory.
Fixes bug #1279
|
|
Patch from Matthew Woehlke. This fixes post-review to work on older p4d (which I
am stuck with at $DAYJOB); without this change post-review can only be used on
submitted changelists. This also adds support for the default changelist (for
all p4/p4d versions).
Reviewed at http://reviews.review-board.org/r/1031/
and http://reviews.review-board.org/r/804/
Fixes bug 1020.
|
|
|
|
|
|
This change introduces Repository UUID support to git-svn, allowing
Review Board to match repositories by UUID instead of comparing the
repository name directly.
Patch by Matthew Woehlke.
Reviewed at http://reviews.review-board.org/r/1030/
|
|
|
|
improperly check for the os.uname function.
Fixes bug #1280.
|
|
This introduces a new column, raw_file_url, to Repository, and shows it in the
admin UI for SCMTools that support it. This URL provides a way for certain
SCMTools (right now, Git only) to access files over HTTP, instead of checking
the file out through the SCM or accessing it locally (in the case of Git).
The raw file URL acts as a URL with fields to be substituted. If <revision>
exists in the URL, it will be replaced with the revision of the file. If
<filename> exists, it will be replaced with the filename.
This also adds a GitHub hosting service and bug tracker to the Repository
admin. Using this, it's trivial to add any GitHub project without knowing the
raw file URL mask.
Reviewed at http://reviews.review-board.org/r/977/
|
|
Corrected hgsubversion support for more recent versions of hgsubversion and add
support for parent branch, guess summary, and guess description. Patch from
Augie Fackler.
Testing done:
Been using it for over a week, haven't seen any major issues.
Reviewed at http://reviews.review-board.org/r/842/
|
|
When a binary file differed, post-review would miss printing a newline after
"Binary Files differ", making the diff invalid for the subsequent file. This
would cause that file's diff to be lost silently when parsed by reviewboard on
the server side. Patch from Chris Trimble.
Reviewed at http://reviews.review-board.org/r/915/
|
|
Solaris has BSD "diff" and GNU "gdiff". Since we use things which are only
present in the GNU userland, we should account for that. This change checks for
SunOS in the output of os.uname and uses gdiff if necessary.
|
|
SCMClients, which makes it easier to update and maintain.
Reviewed at http://reviews.review-board.org/r/882/
|
|
post-review.exe) wrapping it. The official way to install post-review now is by
installing RBTools.
Reviewed at http://reviews.review-board.org/r/916/
|