Page MenuHomePhabricator

Attempt to automatically build XHPAST.
ClosedPublic

Authored by joshuaspence on May 12 2014, 2:31 PM.
Tags
None
Referenced Files
F15510804: D9076.diff
Wed, Apr 16, 8:55 PM
F15447417: D9076.id.diff
Thu, Mar 27, 11:45 PM
F15432634: D9076.diff
Mon, Mar 24, 7:02 PM
F15411435: D9076.id.diff
Wed, Mar 19, 9:17 AM
F15411434: D9076.diff
Wed, Mar 19, 9:17 AM
F15399655: D9076.id21556.diff
Mar 17 2025, 6:27 AM
F15397684: D9076.id.diff
Mar 16 2025, 9:45 PM
F15397404: D9076.id21555.diff
Mar 16 2025, 7:57 PM
Subscribers

Details

Summary

Fixes T4951. Whilst xhpast was originally intended to be an internal development tool, it is not unreasonable for ArcanistXHPASTLinter to be used externally. Currently, however, if the compiled version of xhpast is not up-to-date, we simply exit and ask the user to build XHPAST themselves. Instead, let's try to build XHPAST automatically.

Test Plan

Removed ./src/parser/xhpast/bin/xhpast and ran arc lint over a PHP file. Verified that the xhpast binary was created during this process.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Skipped
Unit
Tests Skipped