Page MenuHomePhabricator

Move `PhutilLibraryMapBuilder` to `libphutil`
ClosedPublic

Authored by joshuaspence on Jul 3 2014, 10:26 PM.
Tags
None
Referenced Files
F13231905: D9813.diff
Tue, May 21, 12:51 AM
F13227639: D9813.id23546.diff
Mon, May 20, 4:33 AM
F13224969: D9813.diff
Sun, May 19, 12:27 PM
F13210810: D9813.diff
Fri, May 17, 5:14 AM
F13194390: D9813.diff
Sun, May 12, 9:32 PM
F13177455: D9813.diff
Wed, May 8, 7:43 PM
Unknown Object (File)
Mon, Apr 29, 3:22 PM
Unknown Object (File)
Wed, Apr 24, 10:24 PM
Subscribers

Details

Summary

Move scripts/phutil_rebuild_map.php and the PhutilLibraryMapBuilder class from rARC to rPHU. The intention here is to create a stronger ArcanistInfrastructureTestCase acceptance test (and the corresponding acceptance tests for rP and rPHU) by regenerating the library map file (to a temporary output) and comparing it to the existing library map file. This will be able to catch the case in which a new class/function has been added to a library, but arc liberate has not been executed.

Test Plan
> ./scripts/phutil_rebuild_map.php src
Finding source files...
Found 344 files.
Loading symbol cache...
Found 344 files in cache.
Building library map...
Writing map...
Done.

Diff Detail

Repository
rPHU libphutil
Branch
liberate
Lint
Lint Passed
SeverityLocationCodeMessage
Advicescripts/phutil_symbols.php:174XHP16TODO Comment
Advicescripts/phutil_symbols.php:258XHP16TODO Comment
Unit
No Test Coverage
Build Status
Buildable 1508
Build 1508: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Move `scripts/phutil_rebuild_map.php` to rPHU.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence edited edge metadata.

Whoops, forgot PhutilLibraryMapBuilder

joshuaspence retitled this revision from Move `scripts/phutil_rebuild_map.php` to rPHU to Move `PhutilLibraryMapBuilder` to `libphutil`.Jul 3 2014, 10:31 PM
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
  • Also scripts/phutil_symbols.php
  • Also resources/php_compat_info.json
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jul 4 2014, 2:54 PM