Changeset View
Changeset View
Standalone View
Standalone View
src/runtime/ArcanistRuntime.php
Show First 20 Lines • Show All 674 Lines • ▼ Show 20 Lines | final class ArcanistRuntime { | ||||
public function getWorkflowStack() { | public function getWorkflowStack() { | ||||
return $this->stack; | return $this->stack; | ||||
} | } | ||||
private function newConduitEngine(ArcanistConfigurationSourceList $config) { | private function newConduitEngine(ArcanistConfigurationSourceList $config) { | ||||
$conduit_uri = $config->getConfig('phabricator.uri'); | $conduit_uri = $config->getConfig('phabricator.uri'); | ||||
if ($conduit_uri === null) { | if ($conduit_uri === null) { | ||||
$conduit_uri = $config->getConfig('default'); | // For now, read this older config from raw storage. There is currently | ||||
// no definition of this option in the "toolsets" config list, and it | |||||
// would be nice to get rid of it. | |||||
$default_list = $config->getStorageValueList('default'); | |||||
if ($default_list) { | |||||
$conduit_uri = last($default_list)->getValue(); | |||||
} | |||||
} | } | ||||
if ($conduit_uri) { | if ($conduit_uri) { | ||||
// Set the URI path to '/api/'. TODO: Originally, I contemplated letting | // Set the URI path to '/api/'. TODO: Originally, I contemplated letting | ||||
// you deploy Phabricator somewhere other than the domain root, but ended | // you deploy Phabricator somewhere other than the domain root, but ended | ||||
// up never pursuing that. We should get rid of all "/api/" silliness | // up never pursuing that. We should get rid of all "/api/" silliness | ||||
// in things users are expected to configure. This is already happening | // in things users are expected to configure. This is already happening | ||||
// to some degree, e.g. "arc install-certificate" does it for you. | // to some degree, e.g. "arc install-certificate" does it for you. | ||||
Show All 33 Lines |