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

Don't advice using --less-context if already used

Browse source 
Test Plan:
  arc diff
  arc diff --less-context

Reviewers: blair

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D2233
This commit is contained in:
vrana 2012-04-13 16:21:23 -07:00
parent ea0f737e85
commit c320e0987d
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/workflow/diff/ArcanistDiffWorkflow.php
View file

@ -733,8 +733,12 @@ EOTEXT
$change_size = number_format($size); $change_size = number_format($size);
$byte_warning = $byte_warning =
"Diff for '{$file_name}' with context is {$change_size} bytes in ". "Diff for '{$file_name}' with context is {$change_size} bytes in ".
"length. Generally, source changes should not be this large. If ". "length. Generally, source changes should not be this large.";
"this file is a huge text file, try using the '--less-context' flag."; if (!$this->getArgument('less-context')) {
$byte_warning .=
" If this file is a huge text file, try using the ".
"'--less-context' flag.";
}
if ($repository_api instanceof ArcanistSubversionAPI) { if ($repository_api instanceof ArcanistSubversionAPI) {
throw new ArcanistUsageException( throw new ArcanistUsageException(
"{$byte_warning} If the file is not a text file, mark it as ". "{$byte_warning} If the file is not a text file, mark it as ".