Conversation
|
Review requested:
|
|
@anonrig looks like there was a breaking change |
|
I will take a look |
|
cc @codebytere Shelley it seems unofficial.gn file will be deleted once this pull-request lands. We need to update update-merve.py file to avoid this. Just FYI in case you've some time (and faster than me) to fix it! |
3c78a76 to
35dc3c1
Compare
|
PS: I've force pushed to remove the deleted files. |
|
The shared builds are failing because Node.js code is no longer compatible with Merve 1.0.0. Can we add some flag on merve that we could use to detect which syntax to use? E.g. #ifdef MERVE_USE_DOT_NAME
#define DOT_NAME(exp) (exp.name)
#else
#define DOT_NAME(exp) (exp)
#endif
exports_set->Add(context, CreateString(isolate, DOT_NAME(exp))).ToLocalChecked(); |
35dc3c1 to
fbd4685
Compare
|
Ended up using EDIT: that being said, it looks like the non-shared build is still failing |
This is an automated update of merve to 1.2.0.