Error Creating bindings with Scons

[AleFachini]

Hello:
I Have been tring to follow the GDNative C++ Example but i ran into some issues.

when runing command:

scons platform=windows headers=../godot_headers generate_bindings=yes

i get this output:

scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
cl /Fosrc\core\AABB.obj /c src\core\AABB.cpp /TP /nologo /EHsc /D_DEBUG /MDd /I. /Igodot_headers /Iinclude /Iinclude\core
AABB.cpp
c:\godotengine\cpp_example\godot-cpp\include\core\String.hpp(4): fatal error C1083: Cannot open include file: 'gdnative/string.h': No such file or directory
scons: *** [src\core\AABB.obj] Error 2
scons: building terminated because of errors.

C:\GodotEngine\cpp_example\godot-cpp>scons platform=windows headers=../godot_headers generate_bindings=yes
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
cl /Fosrc\core\AABB.obj /c src\core\AABB.cpp /TP /nologo /EHsc /D_DEBUG /MDd /I. /Igodot_headers /Iinclude /Iinclude\core
AABB.cpp
c:\godotengine\cpp_example\godot-cpp\include\core\String.hpp(4): fatal error C1083: Cannot open include file: 'gdnative/string.h': No such file or directory
scons: *** [src\core\AABB.obj] Error 2
scons: building terminated because of errors.

I got the same result using MinGW instead of MVSC. And the string.h file is in the path. what im doing wrong?

[BastiaanOlij]

Hi @AleFachini ,

There are still a few pending changes to the documentation awaiting going onto the live pages:
godotengine/godot-docs#1619
The godot headers repository is now located inside of the godot-cpp folder so it may be that the newer scons implementation is clashing with the older instructions that are online. Also this requires that you clone recursively so it may be using the new folder which in turn may be empty.

I also did a video on making a simple GDNative plugin and compiling it, you may find a clue in there:
https://youtu.be/2hK7vOigbLQ
This is using the newer NativeScript 1.1 api in Godot 3.1, if you're going for 3.0 it may not be useful.

[AleFachini]

Hello and thanks, I made it work the next way(windows):

used this link (with linux guide): https://github.com/GodotNativeTools/godot-cpp

Created godot_cpp_example folder, and open CMD in this dir, then:

1. git clone https://github.com/GodotNativeTools/godot-cpp
2. git clone https://github.com/GodotNativeTools/godot_headers
3. copied manually files from ../godot_headers to ../godot-cpp/godot_headers
4. In CMD pointing godot-cpp folder ran:
   C:\GodotEngine\gdnative_cpp_example\godot-cpp>scons godotbinpath="../godot_fork/bin/godot_binary" p=windows headers=../godot_headers generate_bindings=yes

finally got:

scons: done building targets.

all the files went to:
../godot-cpp/include
../godot-cpp/src

The SimpleLibrary folder and subfolders still empty.

NOTE: godotbinpath="../godot_fork/bin/godot_binary" i dont have any folder with godot compilation i'm using windows godot3.0.exe, and is supposed to work without recompiling, is that correct?

[nneesshh]

Hi @AleFachini, These are my steps when using vs2017:
1、generate "godot_api.json" and copy it to "godot_cpp" root:
".\godot.windows.tools.32.exe --gdnative-generate-json-api godot_api.json"

2、copy "godot-master\modules\gdnative\include*" to youre "godot_headers"

3、open "x86 Native Tools Command Prompt for VS 2017" and run "scons" under "godot_cpp" root:
“scons target=release platform=windows bits=32 headers_dir=./godot_headers generate_bindings=yes"

then you would got "libgodot-cpp.windows.release.32.lib" under "godot_cpp\bin"

[BastiaanOlij]

@AleFachini I hope we get to merge nativescript soon because the instructions in the master are outdated, and the instructions in nativescript assume a number of changes to the scons file so they are too new. Finally the docs:
http://docs.godotengine.org/en/3.0/tutorials/plugins/gdnative/gdnative-cpp-example.html
are almost correct and probably should be the instructions you should follow most closely but the aforementioned PR with the latest changes should really be merged.

If you clone like so:
git clone --recursive https://github.com/GodotNativeTools/godot-cpp
It will download the headers automatically.

Also the scons file no longer calls the godot.exe, that is why the godot_api.json is included (which has been renamed to api.json and is now part of Godot_headers but won't be used until nativescript-1.1 is merged). If you need a newer api.json you will have to call godot.exe with the right parameters. Hence godotbinpath is no longer used and will be ignored.

Running scons in the godot-cpp folder only builds the binding files, after you have done that you still need to compile the example itself by running scons.

[BastiaanOlij]

@AleFachini now that nativescript 1.1 has been merged into master, is this now resolved for you?

The instructions in the current readme in the master if you follow them from scratch work fine for me.

[mustafa-archived]

still having the error