Page MenuHomePhabricator

Give "bin/config" a friendlier error message if "local.json" is not writable
ClosedPublic

Authored by epriestley on Sep 3 2019, 7:19 PM.
Tags
None
Referenced Files
F18832467: D20779.id.diff
Sat, Oct 25, 9:57 PM
F18832272: D20779.id49545.diff
Sat, Oct 25, 9:03 PM
F18831917: D20779.id49550.diff
Sat, Oct 25, 4:57 PM
F18830921: D20779.id.diff
Sat, Oct 25, 9:39 AM
F18826943: D20779.diff
Fri, Oct 24, 6:43 AM
F18807555: D20779.diff
Sun, Oct 19, 2:19 AM
F18798739: D20779.diff
Fri, Oct 17, 8:15 AM
F18734021: D20779.id.diff
Tue, Sep 30, 10:46 PM
Subscribers
None

Details

Summary

Ref T13403. We currently emit a useful error message, but it's not tailored and has a stack trace. Since this is a relatively routine error and on the first-time-setup path, tailor it so it's a bit nicer.

Test Plan
  • Ran bin/config set ... with an unwritable "local.json".
  • Ran bin/config set ... normally.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable