HomePhabricator

Use 'git blame --porcelain' for git blame info

Description

Use 'git blame --porcelain' for git blame info

Summary:
This guards against stability issues with the output format of 'git
blame' (such as git config, localization (ref T5554) or future changes).

For example, git config blame.blankboundary true breaks arc cover
before this patch.

Test Plan:

  • Set git config blame.blankboundary true on a test repo.
  • Ran arc cover. It failed with an exception ("Bad blame?").
  • Applied this patch.
  • arc cover works.

Reviewers: Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: Korvin

Maniphest Tasks: T5554

Differential Revision: https://secure.phabricator.com/D13993

Details

Provenance
jbetaAuthored on
epriestleyCommitted on Aug 25 2015, 4:36 PM
epriestleyPushed on Aug 25 2015, 4:36 PM
Reviewer
Blessed Reviewers
Differential Revision
D13993: Use 'git blame --porcelain' for git blame info
Parents
rARC43f8e7eb7119: Recover from PyLint raising messages at character "-1"
Branches
Unknown
Tags
Unknown
Tasks
T5554: arc fails with svn/git/hg non english locales