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
F13168921: D9172.diff
Tue, May 7, 8:02 AM
Unknown Object (File)
Fri, May 3, 8:51 AM
Unknown Object (File)
Tue, Apr 30, 7:55 AM
Unknown Object (File)
Wed, Apr 24, 11:21 PM
Unknown Object (File)
Fri, Apr 12, 1:43 PM
Unknown Object (File)
Thu, Apr 11, 10:08 AM
Unknown Object (File)
Mar 21 2024, 1:56 PM
Unknown Object (File)
Feb 20 2024, 10:49 PM
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
Lint
Lint Skipped
Unit
Tests Skipped

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).