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
F13983128: D8021.id.diff
Sun, Oct 20, 4:18 AM
F13976934: D8021.diff
Fri, Oct 18, 4:22 PM
F13963690: D8021.diff
Tue, Oct 15, 5:19 PM
F13963684: D8021.id.diff
Tue, Oct 15, 5:17 PM
F13963679: D8021.id18142.diff
Tue, Oct 15, 5:16 PM
F13963676: D8021.id18165.diff
Tue, Oct 15, 5:14 PM
Unknown Object (File)
Sep 3 2024, 9:44 AM
Unknown Object (File)
Sep 1 2024, 6:48 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