Page MenuHomePhabricator

Fix `libfbjs`.
ClosedPublic

Authored by joshuaspence on Jun 9 2014, 11:49 PM.
Tags
None
Referenced Files
F13145263: D9446.diff
Fri, May 3, 8:52 AM
F13140003: D9446.diff
Fri, May 3, 3:25 AM
Unknown Object (File)
Mon, Apr 29, 2:42 PM
Unknown Object (File)
Mon, Apr 29, 12:55 AM
Unknown Object (File)
Wed, Apr 24, 10:13 PM
Unknown Object (File)
Sun, Apr 21, 5:25 PM
Unknown Object (File)
Thu, Apr 11, 8:24 AM
Unknown Object (File)
Mar 31 2024, 1:09 PM
Subscribers

Details

Summary

Currently, I am unable to build libfbjs. More specifically, I get the following error:

g++  -fPIC -Wall -DNOT_FBMAKE=1 -ggdb -g -O0 -DDEBUG  -c -o parser.yacc.o parser.yacc.cpp
In file included from parser.hpp:35:0,
                 from parser.yy:23:
parser.yacc.hpp:186:31: error: ‘Node’ has not been declared
 int yyparse (void* yyscanner, Node* root);
                               ^
make: *** [parser.yacc.o] Error 1

It seems that the issue is that in the generated parser.yacc.hpp, Node isn't fully qualified with the correct namespace (i.e. libfbjs::Node). Comparing parser.y from libfbjs with the same file from xhpast, it seems that the namespace is missing from libfbjs.

Test Plan

After this change, I am able to build libfbjs using make.

Diff Detail

Repository
rJX Javelin
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Fix `libfbjs`..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

This version builds fine on my machine too.

This revision is now accepted and ready to land.Jun 10 2014, 2:41 AM
epriestley updated this revision to Diff 22555.

Closed by commit rJXbeb64901b20a (authored by @joshuaspence, committed by @epriestley).