From 4f3dee80e502955f4b90f312dffded5b6bcb316c Mon Sep 17 00:00:00 2001 From: Gareth Evans Date: Tue, 25 Jun 2013 15:19:23 -0700 Subject: [PATCH] Allow passing library name with flag Summary: Enables the process to pass the user input for library name. Fixes T3342 Test Plan: Little help... `arc liberate` doesn't run on windows. Though I saw the flag :) Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Maniphest Tasks: T3342 Differential Revision: https://secure.phabricator.com/D6306 --- src/workflow/ArcanistLiberateWorkflow.php | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/workflow/ArcanistLiberateWorkflow.php b/src/workflow/ArcanistLiberateWorkflow.php index 4df0499d..28d48ccf 100644 --- a/src/workflow/ArcanistLiberateWorkflow.php +++ b/src/workflow/ArcanistLiberateWorkflow.php @@ -45,6 +45,11 @@ EOTEXT "Force the library map to be updated, even in the presence of ". "lint errors.", ), + 'library-name' => array( + 'param' => 'name', + 'help' => + "Use a flag for library name rather than awaiting user input.", + ), 'remap' => array( 'hide' => true, 'help' => @@ -203,7 +208,10 @@ EOTEXT echo "Creating new libphutil library in '{$path}'.\n"; echo "Choose a name for the new library.\n"; do { - $name = phutil_console_prompt('What do you want to name this library?'); + $name = $this->getArgument("library-name"); + if ($name === null) { + $name = phutil_console_prompt('What do you want to name this library?'); + } if (preg_match('/^[a-z-]+$/', $name)) { break; } else {