Page MenuHomePhabricator

Fix `libfbjs`.
ClosedPublic

Authored by joshuaspence on Jun 9 2014, 11:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:24 AM
Unknown Object (File)
Sun, Mar 31, 1:09 PM
Unknown Object (File)
Jan 28 2024, 11:51 AM
Unknown Object (File)
Jan 3 2024, 9:36 PM
Unknown Object (File)
Nov 9 2023, 8:40 PM
Unknown Object (File)
Oct 5 2023, 12:24 AM
Unknown Object (File)
Oct 2 2023, 11:06 AM
Unknown Object (File)
Sep 11 2023, 5:23 AM
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).