firebelley / godot-export

Automatically exports your Godot games.
MIT License
486 stars 59 forks source link

[ERROR] Export for Android not working (ADB and Debug keystore not configured) #40

Closed gumaciel closed 3 years ago

gumaciel commented 3 years ago

ERROR: _fs_changed: Cannot export project with preset 'Android' due to configuration errors: ADB executable not configured in the Editor Settings. Debug keystore not configured in the Editor Settings nor in the preset.

 At: editor/editor_node.cpp:730.

ERROR: _fs_changed: Project export failed with error code 3 for preset 'Android'. At: editor/editor_node.cpp:753. EditorSettings: Save OK! Error: The process '/home/runner/.local/share/godot/godot_executable/godot' failed with exit code 1

workflow: https://github.com/Poing-Studios/Godot-AdMob-Android-iOS/runs/1392365046?check_suite_focus=true

firebelley commented 3 years ago

I'm not too familiar with Android export, but I think you might need additional editor settings? This file https://github.com/firebelley/godot-export/blob/master/dist/editor_settings-3.tres gets copied to the appropriate directory before export happens.

So there are two approaches here:

Let me know if that doesn't make sense, I am happy to explain further!

gumaciel commented 3 years ago

I will try to see this soon, I'm currently working on a mono compiled version, thank you

gumaciel commented 3 years ago

Please approve #41 , worked: https://github.com/Poing-Studios/Godot-AdMob-Android-iOS/actions/runs/373552938, failed due i am trying to test on XCode but it's not complete, thank you.

And i guess theres some way to improve this, because iam generating by myself the debug.keystore and moving the adb to /usr/bin, maybe this is not the best way to do that

firebelley commented 3 years ago

Hi, thanks for the PR. Please look at my comment here https://github.com/firebelley/godot-export/pull/41#issuecomment-731848435

I think it's probably better to allow people to copy a custom editor_settings-3.tres file before running the action, and to not overwrite it if it exists when the action runs. It'll be an extra step in your workflow but I think this solution is going to be the best going forward.

Please let me know if you have further issues!