Merge pull request #79504 from Pingar5/master
Android: Add option to always use WiFi to connect to remote debug
This commit is contained in:
commit
120abd12c2
|
@ -141,7 +141,9 @@ void EditorExportPlatform::gen_debug_flags(Vector<String> &r_flags, int p_flags)
|
|||
String host = EDITOR_GET("network/debug/remote_host");
|
||||
int remote_port = (int)EDITOR_GET("network/debug/remote_port");
|
||||
|
||||
if (p_flags & DEBUG_FLAG_REMOTE_DEBUG_LOCALHOST) {
|
||||
if (EditorSettings::get_singleton()->has_setting("export/android/use_wifi_for_remote_debug") && EDITOR_GET("export/android/use_wifi_for_remote_debug")) {
|
||||
host = EDITOR_GET("export/android/wifi_remote_debug_host");
|
||||
} else if (p_flags & DEBUG_FLAG_REMOTE_DEBUG_LOCALHOST) {
|
||||
host = "localhost";
|
||||
}
|
||||
|
||||
|
|
|
@ -53,6 +53,9 @@ void register_android_exporter() {
|
|||
EDITOR_DEF("export/android/shutdown_adb_on_exit", true);
|
||||
|
||||
EDITOR_DEF("export/android/one_click_deploy_clear_previous_install", false);
|
||||
|
||||
EDITOR_DEF("export/android/use_wifi_for_remote_debug", false);
|
||||
EDITOR_DEF("export/android/wifi_remote_debug_host", "localhost");
|
||||
#endif
|
||||
|
||||
Ref<EditorExportPlatformAndroid> exporter = Ref<EditorExportPlatformAndroid>(memnew(EditorExportPlatformAndroid));
|
||||
|
|
|
@ -1954,8 +1954,9 @@ Error EditorExportPlatformAndroid::run(const Ref<EditorExportPreset> &p_preset,
|
|||
return ERR_SKIP;
|
||||
}
|
||||
|
||||
const bool use_wifi_for_remote_debug = EDITOR_GET("export/android/use_wifi_for_remote_debug");
|
||||
const bool use_remote = (p_debug_flags & DEBUG_FLAG_REMOTE_DEBUG) || (p_debug_flags & DEBUG_FLAG_DUMB_CLIENT);
|
||||
const bool use_reverse = devices[p_device].api_level >= 21;
|
||||
const bool use_reverse = devices[p_device].api_level >= 21 && !use_wifi_for_remote_debug;
|
||||
|
||||
if (use_reverse) {
|
||||
p_debug_flags |= DEBUG_FLAG_REMOTE_DEBUG_LOCALHOST;
|
||||
|
@ -2068,7 +2069,10 @@ Error EditorExportPlatformAndroid::run(const Ref<EditorExportPreset> &p_preset,
|
|||
print_line("Reverse result2: " + itos(rv));
|
||||
}
|
||||
} else {
|
||||
static const char *const msg = "--- Device API < 21; debugging over Wi-Fi ---";
|
||||
static const char *const api_version_msg = "--- Device API < 21; debugging over Wi-Fi ---";
|
||||
static const char *const manual_override_msg = "--- Wi-Fi remote debug enabled in project settings; debugging over Wi-Fi ---";
|
||||
|
||||
const char *const msg = use_wifi_for_remote_debug ? manual_override_msg : api_version_msg;
|
||||
EditorNode::get_singleton()->get_log()->add_message(msg, EditorLog::MSG_TYPE_EDITOR);
|
||||
print_line(String(msg).to_upper());
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue