microsoft / Chakra-Samples

Repository for Chakra JavaScript engine related samples.
MIT License
216 stars 84 forks source link

Linux_OSX_Shared/Sample.cpp Build Error #80

Closed bpavesi closed 7 years ago

bpavesi commented 7 years ago

When i build the sample i get this Error:

g++ sample.cpp -lstdc++ -std=c++0x -I../lib/Jsrt -Wl,--whole-archive ../out/Debug/libChakraCore.so -Wl,--no-whole-archive -pthread -lm -ldl -licuuc -Wno-c++11-compat-deprecated-writable-strings -Wno-deprecated-declarations -Wno-unknown-warning-option -o sample.o sample.cpp: In function ‘int main()’: sample.cpp:61:70: error: too few arguments to function ‘JsErrorCode JsCopyString(JsValueRef, char, size_t, size_t, size_t)’ FAIL_CHECK(JsCopyString(resultJSString, nullptr, 0, &stringLength)); ^ sample.cpp:15:31: note: in definition of macro ‘FAIL_CHECK’ JsErrorCode errCode = cmd; \ ^ In file included from sample.cpp:6:0: ../lib/Jsrt/ChakraCore.h:310:5: note: declared here JsCopyString( ^ sample.cpp:63:81: error: too few arguments to function ‘JsErrorCode JsCopyString(JsValueRef, char, size_t, size_t, size_t)’ FAIL_CHECK(JsCopyString(resultJSString, resultSTR, stringLength + 1, nullptr)); ^ sample.cpp:15:31: note: in definition of macro ‘FAIL_CHECK’ JsErrorCode errCode = cmd; \ ^ In file included from sample.cpp:6:0: ../lib/Jsrt/ChakraCore.h:310:5: note: declared here JsCopyString( ^ At global scope: cc1plus: warning: unrecognized command line option ‘-Wno-unknown-warning-option’ cc1plus: warning: unrecognized command line option ‘-Wno-c++11-compat-deprecated-writable-strings’ Makefile:32: set di istruzioni per l'obiettivo "testmake" non riuscito make: *** [testmake] Errore 1

liminzhu commented 7 years ago

Hey @bpavesi , thanks for the issue! We recently changed the signature of JsCopyString https://github.com/Microsoft/ChakraCore/pull/3433, therefore this sample needs an update. I will get that out shortly.

liminzhu commented 7 years ago

It should be working now with https://github.com/Microsoft/Chakra-Samples/commit/57c761f2a5f0b9737b3b579c78da96fd0e7030a0.

bpavesi commented 7 years ago

it works! tnx

liminzhu commented 7 years ago

Close since it is resolved then.