Page MenuHomePhabricator

Regnerate XHPAST with Bison 3.0.4
ClosedPublic

Authored by epriestley on Apr 29 2016, 1:29 PM.
Tags
None
Referenced Files
F13097843: D15820.diff
Fri, Apr 26, 2:51 AM
Unknown Object (File)
Thu, Apr 25, 12:36 AM
Unknown Object (File)
Wed, Apr 24, 3:37 AM
Unknown Object (File)
Wed, Apr 24, 3:37 AM
Unknown Object (File)
Wed, Apr 24, 3:36 AM
Unknown Object (File)
Wed, Apr 24, 3:36 AM
Unknown Object (File)
Fri, Apr 19, 2:27 AM
Unknown Object (File)
Fri, Apr 19, 2:27 AM
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.