review 966

Usage:

python -m review [options]

Changes:

On August 1st of 2011 Rietveld moved from Subversion to Mercurial. As a result of ommitting some branches, revision number decreased.

966:66d93f2615b7 - 2012-11-30

• Pass the account type through to the HttpRpcServer, by Dominic Hamon.

• Use git similarity option, by Robert Iannucci (6574057, 6726050)

• Fix Removing/updating password in keyring is not possible (#329, 6492080)

• Fix HTTP 403 error on python 2.7 (6490072)

• Fix support for renamed files, aka git mv in upload.py (#251, 4333051)

• Upload binary files that are not images (6221063)

• Add message from upload.py when updating an issue (#351, 5687062)

• Make sure color is not used in git diff (5619043)

• Default –title on –message and clamp at 100 chars (5574050)

801:2f5709156db2 - 2011-01-10 – Options meaning changed

• Subversion 1.7 support, by Jocelyn Fiat (issue #359, 5529052)

• Command line options made intuitive (review 5476044):

  1. on first submission
     -t, --title       issue subject
     -m, --message     issue description
     -F, --file <file> read description from file
  2. on issue update
     -t, --title       new patchset title
     -m, --message     message to reviewers
     -F, --file <file> read message from file

• New upload_complete hook for async processing (review 5440044)

• Print error messages for server errors (review 5399053)

• Fix fail when Hg is executed from subdir (issue #345, 5364065)

• Allow empty files to be uploaded from Git (review 5370042)

709:840f9bb917ba - 2011-09-22

• Add repository ID field to link Issue with Repository (review 5093045)

• More fixes to fit upload.py options in one screen (review 4962070)

695:ba3f47e4a614 - 2011-09-07

• Hide perforce options unless explicitly requested (4968071)

• –send_patch option to attach diff instead of inline, by Kaelyn (4881041)

• Ignore git submodules when generating diffs (issue #324, 4822044)

• Escape @ in filenames when running SVN commands (issue #322, 4745041)

• Remove MIMETYPES whitelist as a way to detect if file is binary (4641078)

Before moving to Mercurial:

r745 - Better handling of login errors, patch by proppy@goog (#289, 4547067)

r730 - Accept ‘cvs diff’ return1 either 0 or 1, by Matthew Dempsky (4435076)

r703 - Make RealMain() suitable for API calls (4439056)

r700 - -r N:M failure on new directories, by Arkadi Shishlov (#187, 4368045)

r687 - Fix upload.py typos, by leeight (#286, 4277081)

r680 - Add CVS backend, by Matthew Dempsky 4280044)

r654 - Preserve username in Base URL until server upload (#266, 4071042)

r640 - Add perforce support, by Alex McCarthy (2635043)

r632 - Support for Rietveld instances moved by HTTP 301 (3227041)

r590 - Look at the old filename only for add with Mercurial (2951041)

r588 - Support “http://” prefix when detecting localhost usage (2736045)

r585 - Fix wrong base revision for Python repository (issue #239, 2442042)

r579 - No depth restriction on ‘hg diff’ by default (issue #234, 2292041)

r566 - Remove “default” from help text for –verbose (2048041)

r561 - Locate subversion config on Windows (1983049)

r554 - Fix –rev option (git backend) when argument has a colon (issue #211)

r552 - Fix exception while trying to output an error (3109015)

r541 - Fix variable usage when getting/setting keyring password (1900041)

r538 - Fetch base file from SVN url if local history is missing (issue #208)

r537 - Optionally use the ‘keyring’ module to avoid password prompts

r535 - Improve captcha message for Google Apps accounts (1690047)

r534 - Convert Unicode values to ASCII (1590044)

r531 - Add account type flag to upload.py (issue #195)

• Home Page: http://codereview.appspot.com/

• Development: http://code.google.com/p/rietveld/

• Discussions: http://groups.google.com/group/codereview-discuss/

• PyPI Code: http://bitbucket.org/techtonik/python-review/