Cleanup of OpenXR module scons config

godotengine · Jan 25, 2024 · 82a36cf · 82a36cf
1 parent 4b6ad34
commit 82a36cf
Show file tree

Hide file tree

Showing 12 changed files with 59 additions and 45 deletions.
diff --git a/modules/openxr/SCsub b/modules/openxr/SCsub
@@ -93,37 +93,13 @@ if env["builtin_openxr"]:
 module_obj = []
 
 env_openxr.add_source_files(module_obj, "*.cpp")
-env_openxr.add_source_files(module_obj, "action_map/*.cpp")
-env_openxr.add_source_files(module_obj, "scene/*.cpp")
+env.modules_sources += module_obj
 
-# We're a little more targeted with our extensions
-if env["platform"] == "android":
- env_openxr.add_source_files(module_obj, "extensions/openxr_android_extension.cpp")
-if env["vulkan"]:
- env_openxr.add_source_files(module_obj, "extensions/openxr_vulkan_extension.cpp")
-if env["opengl3"] and env["platform"] != "macos":
- env_openxr.add_source_files(module_obj, "extensions/openxr_opengl_extension.cpp")
-
-env_openxr.add_source_files(module_obj, "extensions/openxr_local_floor_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_palm_pose_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_composition_layer_depth_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_eye_gaze_interaction.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_htc_controller_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_htc_vive_tracker_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_huawei_controller_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_hand_tracking_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_fb_foveation_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_fb_update_swapchain_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_fb_passthrough_extension_wrapper.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_fb_display_refresh_rate_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_pico_controller_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_wmr_controller_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_ml2_controller_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_extension_wrapper_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_api_extension.cpp")
-env_openxr.add_source_files(module_obj, "extensions/openxr_meta_controller_extension.cpp")
+Export("env_openxr")
 
-env.modules_sources += module_obj
+SConscript("action_map/SCsub")
+SConscript("extensions/SCsub")
+SConscript("scene/SCsub")
 
 if env.editor_build:
  SConscript("editor/SCsub")

diff --git a/modules/openxr/action_map/SCsub b/modules/openxr/action_map/SCsub
@@ -0,0 +1,10 @@
+#!/usr/bin/env python
+
+Import("env")
+Import("env_openxr")
+
+module_obj = []
+
+env_openxr.add_source_files(module_obj, "*.cpp")
+
+env.modules_sources += module_obj
diff --git a/modules/openxr/extensions/SCsub b/modules/openxr/extensions/SCsub
@@ -0,0 +1,18 @@
+#!/usr/bin/env python
+
+Import("env")
+Import("env_openxr")
+
+module_obj = []
+
+env_openxr.add_source_files(module_obj, "*.cpp")
+
+# These are platform dependent
+if env["platform"] == "android":
+ env_openxr.add_source_files(module_obj, "platform/openxr_android_extension.cpp")
+if env["vulkan"]:
+ env_openxr.add_source_files(module_obj, "platform/openxr_vulkan_extension.cpp")
+if env["opengl3"] and env["platform"] != "macos":
+ env_openxr.add_source_files(module_obj, "platform/openxr_opengl_extension.cpp")
+
+env.modules_sources += module_obj
diff --git a/...r/extensions/openxr_android_extension.cpp → ...ons/platform/openxr_android_extension.cpp b/...r/extensions/openxr_android_extension.cpp → ...ons/platform/openxr_android_extension.cpp
@@ -30,7 +30,7 @@
 
 #include "openxr_android_extension.h"
 
-#include "../openxr_api.h"
+#include "../../openxr_api.h"
 
 #include "java_godot_wrapper.h"
 #include "os_android.h"

diff --git a/...nxr/extensions/openxr_android_extension.h → ...sions/platform/openxr_android_extension.h b/...nxr/extensions/openxr_android_extension.h → ...sions/platform/openxr_android_extension.h
@@ -31,8 +31,8 @@
 #ifndef OPENXR_ANDROID_EXTENSION_H
 #define OPENXR_ANDROID_EXTENSION_H
 
-#include "../util.h"
-#include "openxr_extension_wrapper.h"
+#include "../../util.h"
+#include "../openxr_extension_wrapper.h"
 
 class OpenXRAndroidExtension : public OpenXRExtensionWrapper {
 public:

diff --git a/...xr/extensions/openxr_opengl_extension.cpp → ...ions/platform/openxr_opengl_extension.cpp b/...xr/extensions/openxr_opengl_extension.cpp → ...ions/platform/openxr_opengl_extension.cpp
@@ -32,7 +32,7 @@
 
 #ifdef GLES3_ENABLED
 
-#include "../openxr_util.h"
+#include "../../openxr_util.h"
 
 #include "drivers/gles3/effects/copy_effects.h"
 #include "drivers/gles3/storage/texture_storage.h"

diff --git a/...enxr/extensions/openxr_opengl_extension.h → ...nsions/platform/openxr_opengl_extension.h b/...enxr/extensions/openxr_opengl_extension.h → ...nsions/platform/openxr_opengl_extension.h
@@ -33,14 +33,14 @@
 
 #ifdef GLES3_ENABLED
 
-#include "../openxr_api.h"
-#include "../util.h"
-#include "openxr_extension_wrapper.h"
+#include "../../openxr_api.h"
+#include "../../util.h"
+#include "../openxr_extension_wrapper.h"
 
 #include "core/templates/vector.h"
 
 // Always include this as late as possible.
-#include "../openxr_platform_inc.h"
+#include "../../openxr_platform_inc.h"
 
 class OpenXROpenGLExtension : public OpenXRGraphicsExtensionWrapper {
 public:

diff --git a/...xr/extensions/openxr_vulkan_extension.cpp → ...ions/platform/openxr_vulkan_extension.cpp b/...xr/extensions/openxr_vulkan_extension.cpp → ...ions/platform/openxr_vulkan_extension.cpp
@@ -30,7 +30,7 @@
 
 #include "openxr_vulkan_extension.h"
 
-#include "../openxr_util.h"
+#include "../../openxr_util.h"
 
 #include "core/string/print_string.h"
 #include "servers/rendering/renderer_rd/effects/copy_effects.h"

diff --git a/...enxr/extensions/openxr_vulkan_extension.h → ...nsions/platform/openxr_vulkan_extension.h b/...enxr/extensions/openxr_vulkan_extension.h → ...nsions/platform/openxr_vulkan_extension.h
@@ -31,14 +31,14 @@
 #ifndef OPENXR_VULKAN_EXTENSION_H
 #define OPENXR_VULKAN_EXTENSION_H
 
-#include "../openxr_api.h"
-#include "../util.h"
-#include "openxr_extension_wrapper.h"
+#include "../../openxr_api.h"
+#include "../../util.h"
+#include "../openxr_extension_wrapper.h"
 
 #include "core/templates/vector.h"
 
 // Always include this as late as possible.
-#include "../openxr_platform_inc.h"
+#include "../../openxr_platform_inc.h"
 
 class OpenXRVulkanExtension : public OpenXRGraphicsExtensionWrapper, VulkanHooks {
 public:

diff --git a/modules/openxr/openxr_api.cpp b/modules/openxr/openxr_api.cpp
@@ -46,11 +46,11 @@
 #include "openxr_platform_inc.h"
 
 #ifdef VULKAN_ENABLED
-#include "extensions/openxr_vulkan_extension.h"
+#include "extensions/platform/openxr_vulkan_extension.h"
 #endif
 
 #if defined(GLES3_ENABLED) && !defined(MACOS_ENABLED)
-#include "extensions/openxr_opengl_extension.h"
+#include "extensions/platform/openxr_opengl_extension.h"
 #endif
 
 #include "extensions/openxr_composition_layer_depth_extension.h"

diff --git a/modules/openxr/register_types.cpp b/modules/openxr/register_types.cpp
@@ -61,7 +61,7 @@
 #endif
 
 #ifdef ANDROID_ENABLED
-#include "extensions/openxr_android_extension.h"
+#include "extensions/platform/openxr_android_extension.h"
 #endif
 
 #include "core/config/project_settings.h"

diff --git a/modules/openxr/scene/SCsub b/modules/openxr/scene/SCsub
@@ -0,0 +1,10 @@
+#!/usr/bin/env python
+
+Import("env")
+Import("env_openxr")
+
+module_obj = []
+
+env_openxr.add_source_files(module_obj, "*.cpp")
+
+env.modules_sources += module_obj