Page MenuHomePhabricator

Regnerate XHPAST with Bison 3.0.4
ClosedPublic

Authored by epriestley on Apr 29 2016, 1:29 PM.
Tags
None
Referenced Files
F15471007: D15820.id38110.diff
Sat, Apr 5, 2:54 AM
F15464878: D15820.id38110.diff
Wed, Apr 2, 12:59 PM
F15458464: D15820.id38108.diff
Mon, Mar 31, 3:26 AM
F15447276: D15820.id.diff
Thu, Mar 27, 10:50 PM
F15442400: D15820.diff
Wed, Mar 26, 11:14 PM
F15433992: D15820.id38110.diff
Tue, Mar 25, 1:27 AM
F15431685: D15820.id38108.diff
Mon, Mar 24, 1:56 PM
F15427108: D15820.id.diff
Sun, Mar 23, 12:54 PM
Subscribers
None

Details

Summary

See rARC3ffed59. I think that test failure is some sort of weird parser generator bug in Bison 3.0.2 that appears to affect Ubuntu and OS X, at a minimum.

I don't immediately see what the acutal issue is, nor can I find anything suspicious in the changelog. However, I can reproduce the issue reliably locally (OS X).

If I do this:

$ git checkout master
$ git clean -df
$ ./scripts/build_xhpast.php

...I get test failures in arc.

If I do this:

$ git checkout bison304 # This change
$ git clean -df
$ ./scripts/build_xhpast.php

...I get clean tests.

The actual AST/parser look fine to me from careful inspection, too.

Test Plan

Added a now-passing unit test.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Regnerate XHPAST with Bison 3.0.4.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: richardvanvelzen, chad.

Underlying change was:

  • Install binson version 3.0.4.
  • Run make cleanall.
  • Run make.

Spooky!

iiam

This revision is now accepted and ready to land.Apr 29 2016, 1:46 PM
This revision was automatically updated to reflect the committed changes.