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
F13310791: D8021.diff
Mon, Jun 10, 6:50 PM
F13279980: D8021.id.diff
Sun, Jun 2, 1:53 AM
F13269514: D8021.id18165.diff
Wed, May 29, 6:24 AM
F13267294: D8021.diff
Wed, May 29, 3:07 AM
F13252724: D8021.diff
Sat, May 25, 2:00 AM
F13231114: D8021.diff
Tue, May 21, 12:12 AM
F13210326: D8021.diff
Fri, May 17, 4:43 AM
F13194098: D8021.diff
May 12 2024, 9:18 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