Page MenuHomePhabricator

Improve 'arc get-config', 'arc set-config' and show more config info with --trace
ClosedPublic

Authored by epriestley on May 18 2014, 12:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 27, 3:45 AM
Unknown Object (File)
Sat, Dec 7, 10:23 PM
Unknown Object (File)
Sat, Dec 7, 10:19 AM
Unknown Object (File)
Wed, Dec 4, 9:59 PM
Unknown Object (File)
Wed, Dec 4, 9:59 PM
Unknown Object (File)
Wed, Dec 4, 9:59 PM
Unknown Object (File)
Wed, Dec 4, 9:48 PM
Unknown Object (File)
Nov 25 2024, 8:51 AM
Subscribers

Details

Summary

Fixes T4952. Several issues:

  • You review configuration values with arc set-config --show. This makes no sense and never has, I think it just predated arc get-config or was easier or something.
    • Instead, review values with arc get-config and review details with arc get-config --verbose.
  • Show better and more detailed information about all config sources.
  • Establish and show default values from a new "default" source.
  • With --trace include more information about attempts to read configuration files.
Test Plan

Ran arc get-config --trace --verbose in various working directories and received sensible-looking output.

Screen_Shot_2014-05-17_at_5.49.45_PM.png (310×590 px, 33 KB)

Diff Detail

Repository
rARC Arcanist
Branch
gsconfig
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 538
Build 538: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Improve 'arc get-config', 'arc set-config' and show more config info with --trace.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.May 19 2014, 6:20 PM
epriestley updated this revision to Diff 21895.

Closed by commit rARC104219dd62ee (authored by @epriestley).