Page MenuHomePhabricator

Improve symbol generation scripts
ClosedPublic

Authored by joshuaspence on Aug 15 2018, 1:25 AM.
Tags
None
Referenced Files
F19160465: D19588.id46820.diff
Dec 15 2025, 4:04 PM
F19157440: D19588.id46820.diff
Dec 13 2025, 4:05 PM
F19153365: D19588.id46820.diff
Dec 11 2025, 9:14 AM
F18946736: D19588.diff
Nov 11 2025, 8:49 PM
F18877096: D19588.id.diff
Nov 6 2025, 6:05 AM
F18874158: D19588.diff
Nov 5 2025, 1:23 PM
F18842967: D19588.id46820.diff
Oct 28 2025, 8:01 PM
F18814776: D19588.diff
Oct 20 2025, 11:54 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.