Skip to content
This repository has been archived by the owner on Jun 13, 2024. It is now read-only.

Linux_OSX_Shared/Sample.cpp Build Error #80

Closed
bpavesi opened this issue Aug 22, 2017 · 4 comments
Closed

Linux_OSX_Shared/Sample.cpp Build Error #80

bpavesi opened this issue Aug 22, 2017 · 4 comments

Comments

@bpavesi
Copy link

bpavesi commented Aug 22, 2017

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
Copy link
Member

Hey @bpavesi , thanks for the issue! We recently changed the signature of JsCopyString chakra-core/ChakraCore#3433, therefore this sample needs an update. I will get that out shortly.

@liminzhu
Copy link
Member

It should be working now with 57c761f.

@bpavesi
Copy link
Author

bpavesi commented Aug 23, 2017

it works!
tnx

@liminzhu
Copy link
Member

liminzhu commented Sep 3, 2017

Close since it is resolved then.

@liminzhu liminzhu closed this as completed Sep 3, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants