Page MenuHomePhabricator

Improve symbol generation scripts
ClosedPublic

Authored by joshuaspence on Aug 15 2018, 1:25 AM.
Tags
None
Referenced Files
F18946736: D19588.diff
Tue, Nov 11, 8:49 PM
F18877096: D19588.id.diff
Thu, Nov 6, 6:05 AM
F18874158: D19588.diff
Wed, Nov 5, 1:23 PM
F18842967: D19588.id46820.diff
Tue, Oct 28, 8:01 PM
F18814776: D19588.diff
Mon, Oct 20, 11:54 PM
F18755570: D19588.id.diff
Oct 5 2025, 6:20 AM
F18753116: D19588.id.diff
Oct 4 2025, 5:28 PM
F18751857: D19588.diff
Oct 4 2025, 1:01 PM
Subscribers

Details

Summary

Currently the symbol generation scripts fail if passed a list containing no files because explode("\n", $input) returns array("") rather than array(). This means that a generic Harbormaster Build Plan with a step which executes find . -type f -name '*.php' | ./scripts/generate_php_symbols.php won't work because it fails in repositories that don't contain any PHP code.

Test Plan

Ran echo | generate_php_symbols and saw no output instead of an exception.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

These scripts may not be long for this world (see T13047) but this change seems reasonable.

This revision is now accepted and ready to land.Aug 15 2018, 4:58 PM
This revision was automatically updated to reflect the committed changes.