Page MenuHomePhabricator

Revert xhpast changes that impacted builds under Bison 2.3
ClosedPublic

Authored by epriestley on Apr 7 2020, 9:02 PM.
Tags
None
Referenced Files
F13055813: D21063.id50188.diff
Fri, Apr 19, 12:58 PM
F13053841: D21063.id.diff
Fri, Apr 19, 11:33 AM
Unknown Object (File)
Sat, Mar 30, 6:57 AM
Unknown Object (File)
Tue, Mar 26, 6:44 AM
Unknown Object (File)
Tue, Mar 26, 2:27 AM
Unknown Object (File)
Mar 20 2024, 10:37 PM
Unknown Object (File)
Mar 20 2024, 12:04 PM
Unknown Object (File)
Feb 29 2024, 10:39 PM
Subscribers
None

Details

Summary

Fixes T9753. Changes some time ago (in D13970 + D13974) improved XHPAST build compile-time warning behavior under Bison 3.

However, macOS still ships with Bison 2.3 and these changes prevent XHPAST from building with Bison 2.3. The changes didn't introduce version detection, so Bison 2.3 builds fail somewhat mysteriously without obvious next steps.

It's relatively easy to install Bison 3 on macOS via Homebrew, but the Bison 3 changes aren't terribly substantive and XHPAST doesn't actually depend on any Bison 3 features, so just return to Bison 2.3 for now.

It would be reasonable to undo this again and retarget Bison 3 in the future, but ideally we should wait until macOS ships with Bison 3 or we have a specific reason to bump the minimum required version to 3. If/when we do, we should version-detect Bison and raise a clear error message.

Test Plan

Built xhpast under Bison 2.3 on a default macOS install using "make cleanall && make install".

Diff Detail

Repository
rARC Arcanist
Branch
xhpast1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 24001
Build 33032: Run Core Tests
Build 33031: arc lint + arc unit