Paths

Table of Contentst

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

Authored by epriestley on May 13 2014, 6:31 PM.

Tags

None

Referenced Files

	F15507782: D9105.diff
	Tue, Apr 15, 10:48 PM

	F15491733: D9105.id21621.diff
	Sat, Apr 12, 6:51 AM

	F15484298: D9105.id21621.diff
	Wed, Apr 9, 5:30 PM

	F15484133: D9105.id21622.diff
	Wed, Apr 9, 4:31 PM

	F15474749: D9105.diff
	Sun, Apr 6, 12:00 PM

	F15472099: D9105.id.diff
	Sat, Apr 5, 10:53 AM

	F15469865: D9105.diff
	Fri, Apr 4, 5:28 PM

	F15438080: D9105.id21622.diff
	Tue, Mar 25, 11:17 PM

View All Files

Subscribers

davedash

epriestley

Details

Reviewers

btrahan
davedash

Commits

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

Summary

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

Repository

rARC Arcanist

Branch

default

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 439
Build 439: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 21621.May 13 2014, 6:31 PM

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.

Herald added a subscriber: epriestley. · View Herald TranscriptMay 13 2014, 6:31 PM

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

Harbormaster completed remote builds in B439: Diff 21621.May 13 2014, 6:32 PM

Looks good to me.

This revision is now accepted and ready to land.May 13 2014, 6:36 PM

Closed by commit rARC85b6c581e11b (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

scripts/

arcanist.php

10 lines

Diff 21621

View Options

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