Page MenuHomePhabricator

Make directory fixture script slightly more modern and usable
ClosedPublic

Authored by epriestley on Jan 21 2014, 3:34 PM.
Tags
None
Referenced Files
F15507929: D8021.diff
Wed, Apr 16, 12:20 AM
F15491112: D8021.id18142.diff
Fri, Apr 11, 11:42 PM
F15470899: D8021.diff
Sat, Apr 5, 2:00 AM
F15434755: D8021.id18142.diff
Tue, Mar 25, 5:12 AM
F15427506: D8021.id18142.diff
Sun, Mar 23, 2:44 PM
F15408442: D8021.id.diff
Tue, Mar 18, 11:05 PM
F15406648: D8021.id18165.diff
Tue, Mar 18, 1:07 PM
F15401913: D8021.diff
Mar 17 2025, 7:46 PM
Subscribers

Details

Summary

Ref T4327. I made some small changes to this script while developing parser unit tests. Mostly:

  • Add pht().
  • Use PhutilConsole.
  • Throw PhutilArugmentUsageException for questionable usage.
  • Add a --read-only flag to ignore changes on exit.
  • Show the CWD after unpacking the fixture, which makes it easier to svn checkout file://... it.
  • Improve some language.
Test Plan

Tested all the features; used these features while working on T4327 to good effect.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped