Page MenuHomePhabricator

Fix `libfbjs`.
ClosedPublic

Authored by joshuaspence on Jun 9 2014, 11:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 3, 9:03 AM
Unknown Object (File)
Wed, Nov 27, 1:53 AM
Unknown Object (File)
Fri, Nov 22, 11:30 PM
Unknown Object (File)
Tue, Nov 19, 4:25 PM
Unknown Object (File)
Mon, Nov 18, 7:34 PM
Unknown Object (File)
Mon, Nov 18, 6:53 PM
Unknown Object (File)
Fri, Nov 15, 2:49 AM
Unknown Object (File)
Nov 10 2024, 5:49 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).