Page MenuHomePhabricator

Attempt to automatically build XHPAST.
ClosedPublic

Authored by joshuaspence on May 12 2014, 2:31 PM.
Tags
None
Referenced Files
F13187820: D9076.diff
Sat, May 11, 4:46 AM
F13181325: D9076.id21556.diff
Thu, May 9, 11:47 AM
F13181324: D9076.id21555.diff
Thu, May 9, 11:47 AM
F13181288: D9076.id.diff
Thu, May 9, 11:01 AM
F13181273: D9076.diff
Thu, May 9, 10:49 AM
Unknown Object (File)
Thu, May 2, 9:14 AM
Unknown Object (File)
Wed, Apr 24, 9:59 PM
Unknown Object (File)
Fri, Apr 19, 2:42 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
Branch
autobuild-xhpast
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 400
Build 400: [Placeholder Plan] Wait for 30 Seconds