revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-12-23 05:50:54 +01:00
Code Issues Releases Wiki Activity

Fix error behavior of "arc version" when it encounters a library which is not a working copy

Browse source 
Summary:
Ref T13504. The API has changed here slightly, and if you run "arc version" without "arcanist/" being a Git working copy, it currently fatals in a misleading way.

Instead, reach the error properly.

Test Plan: Ran "arc version" after moving aside ".git/", got a helpful error message instead of a confusing "call on null" exception.

Maniphest Tasks: T13504

Differential Revision: https://secure.phabricator.com/D21050
This commit is contained in:
epriestley 2020-04-02 08:14:18 -07:00
parent 63276697eb
commit 5ce1d79717
1 changed files with 14 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
src/toolset/workflow/ArcanistVersionWorkflow.php
View file

@ -48,15 +48,23 @@ EOTEXT
);
foreach ($roots as $lib => $root) {
$working_copy = ArcanistWorkingCopy::newFromWorkingDirectory($root);
$repository_api = $working_copy->newRepositoryAPI();
$is_git = false;
if (!$repository_api instanceof ArcanistGitAPI) {
throw new ArcanistUsageException(
$working_copy = ArcanistWorkingCopy::newFromWorkingDirectory($root);
if ($working_copy) {
$repository_api = $working_copy->newRepositoryAPI();
if ($repository_api instanceof ArcanistGitAPI) {
$is_git = true;
}
}
if (!$is_git) {
throw new PhutilArgumentUsageException(
pht(
'Library "%s" is not a Git working copy, so no version '.
'Library "%s" (at "%s") is not a Git working copy, so no version '.
'information can be provided.',
$lib));
$lib,
Filesystem::readablePath($root)));
}
// NOTE: Carefully execute these commands in a way that works on Windows