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

Support 'arc:exec(command)' in base-commit DSL

Browse source 
Summary: Allow users to run a command to determine the base revision of the commit range.

Test Plan: Ran stuff like `arc which --show-base --base 'arc:exec(ls)'` and got the expected results.

Reviewers: dschleimer, csilvers, btrahan, vrana

Reviewed By: csilvers

CC: aran

Maniphest Tasks: T1233

Differential Revision: https://secure.phabricator.com/D2830
This commit is contained in:
epriestley 2012-06-22 08:13:06 -07:00
parent 7f640b9db9
commit 887c3484a7
1 changed files with 14 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/parser/ArcanistBaseCommitParser.php
View file

@ -32,7 +32,7 @@ final class ArcanistBaseCommitParser {
return array(); return array();
} }
$spec = preg_split('/[ ,]/', $raw_spec); $spec = preg_split('/\s*,\s*/', $raw_spec);
$spec = array_filter($spec); $spec = array_filter($spec);
foreach ($spec as $rule) { foreach ($spec as $rule) {
@ -164,6 +164,19 @@ final class ArcanistBaseCommitParser {
case 'outgoing': case 'outgoing':
return $this->api->resolveBaseCommitRule($rule, $source); return $this->api->resolveBaseCommitRule($rule, $source);
default: default:
$matches = null;
if (preg_match('/^exec\((.*)\)$/', $name, $matches)) {
$root = $this->api->getWorkingCopyIdentity()->getProjectRoot();
$future = new ExecFuture($matches[1]);
$future->setCWD($root);
list($err, $stdout) = $future->resolve();
if (!$err) {
return trim($stdout);
} else {
return null;
}
}
throw new ArcanistUsageException( throw new ArcanistUsageException(
"Base commit rule '{$rule}' (from source '{$source}') ". "Base commit rule '{$rule}' (from source '{$source}') ".
"is not a recognized rule."); "is not a recognized rule.");