Page MenuHomePhabricator

Read 'phabricator.uri' and 'default' out of more config sources

Authored by epriestley on May 13 2014, 6:31 PM.
Referenced Files
F13259830: D9105.diff
Sun, May 26, 11:09 PM
F13255929: D9105.diff
Sat, May 25, 9:11 AM
F13242716: D9105.diff
Thu, May 23, 3:02 AM
F13242221: D9105.id21622.diff
Thu, May 23, 2:04 AM
F13242220: D9105.id21621.diff
Thu, May 23, 2:04 AM
Thu, May 23, 2:04 AM
F13220742: D9105.diff
Sun, May 19, 1:54 AM
F13202748: D9105.diff
Tue, May 14, 11:05 PM



We have old/inconsistent code for reading these. Instead, read from sources in a modern way.

Test Plan
  • Removed "default" and "phabricator.uri" from user/local/project config.
  • Set "default" in "/etc/arcconfig".
  • Ran arc tasks and got a "connect to Phabricator" message.

Diff Detail

rARC Arcanist
Lint Skipped
Tests Skipped

Event Timeline

epriestley retitled this revision from to Read 'phabricator.uri' and 'default' out of more config sources.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.

Specifically, "default" and "phabricator.uri" currently have no effect in system configuration.

davedash added a reviewer: davedash.
davedash added a subscriber: davedash.

Looks good to me.

This revision is now accepted and ready to land.May 13 2014, 6:36 PM
epriestley updated this revision to Diff 21622.

Closed by commit rARC85b6c581e11b (authored by @epriestley).