Page MenuHomePhabricator

Attempt to automatically build XHPAST.
ClosedPublic

Authored by joshuaspence on May 12 2014, 2:31 PM.
Tags
None
Referenced Files
F15411435: D9076.id.diff
Wed, Mar 19, 9:17 AM
F15411434: D9076.diff
Wed, Mar 19, 9:17 AM
F15399655: D9076.id21556.diff
Mon, Mar 17, 6:27 AM
F15397684: D9076.id.diff
Sun, Mar 16, 9:45 PM
F15397404: D9076.id21555.diff
Sun, Mar 16, 7:57 PM
F15396992: D9076.id21556.diff
Sun, Mar 16, 5:26 PM
F15394166: D9076.diff
Sat, Mar 15, 11:21 PM
F15393624: D9076.id21556.diff
Sat, Mar 15, 10:53 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