revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-22 14:52:40 +01:00
Code Issues Releases Wiki Activity

Remove "--no-decorate"

Browse source 
Summary: Apparently my advice here was terrible, and `--no-decorate` and `--decorate=no` are both very recent additions to Git which a bunch of users don't have. Get rid of them since D2344 allows us to parse all decorate levels anyway.

Test Plan: Tried to google "git changelog", got a bunch of pages about managing changelogs with git.

Reviewers: zeeg, ehren

Reviewed By: ehren

CC: ehren, aran, NorthIsUp

Differential Revision: https://secure.phabricator.com/D2354
This commit is contained in:
epriestley 2012-04-30 17:28:04 -07:00
parent 7070d0a065
commit c0c3c8a113
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/repository/api/git/ArcanistGitAPI.php
View file

@ -292,11 +292,11 @@ final class ArcanistGitAPI extends ArcanistRepositoryAPI {
} else if ($relative == self::GIT_MAGIC_ROOT_COMMIT) {
// First commit.
list($stdout) = $this->execxLocal(
'log --no-decorate --format=medium HEAD');
'log --format=medium HEAD');
} else {
// 2..N commits.
list($stdout) = $this->execxLocal(
'log --no-decorate --first-parent --format=medium %s..HEAD',
'log --first-parent --format=medium %s..HEAD',
$this->getRelativeCommit());
}
return $stdout;