Proper dependencies to LmbrCentral/LmbrCentral.Editor, mixing those two can cause Editor modules to load non-Editor modules

5 years ago · db0db5c1c0
parent bff7d39f68
commit db0db5c1c0
14 changed files with 22 additions and 25 deletions
--- a/Code/Sandbox/Plugins/ComponentEntityEditorPlugin/CMakeLists.txt
+++ b/Code/Sandbox/Plugins/ComponentEntityEditorPlugin/CMakeLists.txt
@ -35,7 +35,7 @@ ly_add_target(
            AZ::AzToolsFramework
            Legacy::CryCommon
            Legacy::EditorLib
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Editor
 )

 ly_add_dependencies(Editor ComponentEntityEditorPlugin)
@ -65,7 +65,7 @@ if(PAL_TRAIT_BUILD_TESTS_SUPPORTED)
                AZ::AzToolsFrameworkTestCommon
                Legacy::CryCommon
                Legacy::EditorLib
-                Gem::LmbrCentral
+                Gem::LmbrCentral.Editor
    )
    ly_add_googletest(
        NAME Legacy::ComponentEntityEditorPlugin.Tests
--- a/Gems/AutomatedLauncherTesting/Code/CMakeLists.txt
+++ b/Gems/AutomatedLauncherTesting/Code/CMakeLists.txt
@ -23,7 +23,7 @@ ly_add_target(
        PUBLIC
            AZ::AzCore
            Legacy::CryCommon
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
 )

 ly_add_target(
--- a/Gems/EMotionFX/Code/CMakeLists.txt
+++ b/Gems/EMotionFX/Code/CMakeLists.txt
@ -36,7 +36,7 @@ ly_add_target(
            AZ::AzCore
            AZ::AzFramework
            Legacy::CryCommon
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
        PUBLIC
            AZ::AtomCore
            Gem::Atom_RPI.Public
--- a/Gems/FastNoise/Code/CMakeLists.txt
+++ b/Gems/FastNoise/Code/CMakeLists.txt
@ -23,7 +23,7 @@ ly_add_target(
        PUBLIC
            Legacy::CryCommon
            Gem::GradientSignal
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
 )

 ly_add_target(
--- a/Gems/GradientSignal/Code/CMakeLists.txt
+++ b/Gems/GradientSignal/Code/CMakeLists.txt
@ -22,7 +22,7 @@ ly_add_target(
    BUILD_DEPENDENCIES
        PUBLIC
            Legacy::CryCommon
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
            Gem::SurfaceData
            Gem::ImageProcessingAtom.Headers
 )
@ -67,7 +67,7 @@ if(PAL_TRAIT_BUILD_HOST_TOOLS)
                3rdParty::Qt::Widgets
                Legacy::CryCommon
                AZ::AzToolsFramework
-                Gem::LmbrCentral
+                Gem::LmbrCentral.Editor
                Gem::SurfaceData
                AZ::AssetBuilderSDK
                Gem::GradientSignal.Static
--- a/Gems/ImGui/Code/CMakeLists.txt
+++ b/Gems/ImGui/Code/CMakeLists.txt
@ -70,7 +70,7 @@ ly_add_target(
    BUILD_DEPENDENCIES
        PUBLIC
            Gem::ImGui.ImGuiLYUtils
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
 )

 ly_add_target(
--- a/Gems/LandscapeCanvas/Code/CMakeLists.txt
+++ b/Gems/LandscapeCanvas/Code/CMakeLists.txt
@ -35,7 +35,7 @@ if(PAL_TRAIT_BUILD_HOST_TOOLS)
                Legacy::CryCommon
                Legacy::Editor.Headers
                Legacy::EditorCommon
-                Gem::LmbrCentral
+                Gem::LmbrCentral.Editor
                Gem::GraphCanvasWidgets
                Gem::GraphModel.Editor.Static
                Gem::GradientSignal.Editor
--- a/Gems/LyShine/Code/CMakeLists.txt
+++ b/Gems/LyShine/Code/CMakeLists.txt
@ -26,13 +26,13 @@ ly_add_target(
    BUILD_DEPENDENCIES
        PRIVATE
            Legacy::CryCommon
-            Gem::LmbrCentral
-            Gem::TextureAtlas
        PUBLIC
            Gem::Atom_RPI.Public
            Gem::Atom_Utils.Static
            Gem::Atom_Bootstrap.Headers
            Gem::AtomFont
+            Gem::LmbrCentral.Static
+            Gem::TextureAtlas
 )

 ly_add_target(
@ -49,8 +49,6 @@ ly_add_target(
        PRIVATE
            Gem::LyShine.Static
            Legacy::CryCommon
-            Gem::LmbrCentral
-            Gem::TextureAtlas
    RUNTIME_DEPENDENCIES
        Gem::LmbrCentral
        Gem::TextureAtlas
@ -85,7 +83,7 @@ if (PAL_TRAIT_BUILD_HOST_TOOLS)
                Legacy::EditorCore
                Gem::LyShine.Static
                Legacy::CryCommon
-                Gem::LmbrCentral
+                Gem::LmbrCentral.Editor.Static
                Gem::TextureAtlas
                Gem::AtomToolsFramework.Static
                Gem::AtomToolsFramework.Editor
@ -117,7 +115,7 @@ if (PAL_TRAIT_BUILD_HOST_TOOLS)
                Legacy::CryCommon
                AZ::AssetBuilderSDK
                Gem::LyShine.Editor.Static
-                Gem::LmbrCentral
+                Gem::LmbrCentral.Editor
                Gem::TextureAtlas
        RUNTIME_DEPENDENCIES
            Gem::LmbrCentral.Editor
@ -175,7 +173,7 @@ if(PAL_TRAIT_BUILD_TESTS_SUPPORTED)
                    AZ::AzTest
                    Legacy::CryCommon
                    AZ::AssetBuilderSDK
-                    Gem::LmbrCentral
+                    Gem::LmbrCentral.Editor
                    Gem::TextureAtlas
                    Gem::LyShine.Editor.Static
        )
--- a/Gems/LyShineExamples/Code/CMakeLists.txt
+++ b/Gems/LyShineExamples/Code/CMakeLists.txt
@ -22,7 +22,7 @@ ly_add_target(
    BUILD_DEPENDENCIES
        PUBLIC
            Legacy::CryCommon
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
            Gem::LyShine.Static
 )

@ -39,4 +39,6 @@ ly_add_target(
    BUILD_DEPENDENCIES
        PRIVATE
            Gem::LyShineExamples.Static
+    RUNTIME_DEPENDENCIES
+        Gem::LmbrCentral
 )
--- a/Gems/Maestro/Code/CMakeLists.txt
+++ b/Gems/Maestro/Code/CMakeLists.txt
@ -22,7 +22,6 @@ ly_add_target(
    BUILD_DEPENDENCIES
        PRIVATE
            Legacy::CryCommon
-            Gem::LmbrCentral
 )

 ly_add_target(
@ -39,7 +38,6 @@ ly_add_target(
        PRIVATE
            Legacy::CryCommon
            Gem::Maestro.Static
-            Gem::LmbrCentral
    RUNTIME_DEPENDENCIES
        Gem::LmbrCentral
 )
@ -69,7 +67,6 @@ if (PAL_TRAIT_BUILD_HOST_TOOLS)
                AZ::AzToolsFramework
                AZ::AssetBuilderSDK
                Gem::Maestro.Static
-                Gem::LmbrCentral
        RUNTIME_DEPENDENCIES
            Gem::LmbrCentral.Editor
    )
--- a/Gems/PhysX/Code/CMakeLists.txt
+++ b/Gems/PhysX/Code/CMakeLists.txt
@ -46,7 +46,7 @@ ly_add_target(
            AZ::AzCore
            AZ::AzFramework
            Legacy::CryCommon
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
 )

 ly_add_target(
@ -107,7 +107,7 @@ if(PAL_TRAIT_BUILD_HOST_TOOLS)
                AZ::SceneCore
                AZ::SceneData
                Legacy::CryCommon
-                Gem::LmbrCentral
+                Gem::LmbrCentral.Editor.Static
                Gem::PhysX.NumericalMethods
                Gem::PhysX.Static
                Gem::AtomLyIntegration_CommonFeatures.Static
--- a/Gems/StartingPointCamera/Code/CMakeLists.txt
+++ b/Gems/StartingPointCamera/Code/CMakeLists.txt
@ -23,7 +23,7 @@ ly_add_target(
        PRIVATE
            AZ::AzCore
            Gem::CameraFramework.Static
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
            Legacy::CryCommon
 )

--- a/Gems/SurfaceData/Code/CMakeLists.txt
+++ b/Gems/SurfaceData/Code/CMakeLists.txt
@ -25,7 +25,7 @@ ly_add_target(
        PUBLIC
            Gem::Atom_RPI.Public
            Gem::Atom_Feature_Common.Static
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
 )

 ly_add_target(
--- a/Gems/Vegetation/Code/CMakeLists.txt
+++ b/Gems/Vegetation/Code/CMakeLists.txt
@ -26,7 +26,7 @@ ly_add_target(
    BUILD_DEPENDENCIES
        PUBLIC
            Legacy::CryCommon
-            Gem::LmbrCentral
+            Gem::LmbrCentral.Static
            Gem::GradientSignal
            Gem::SurfaceData.Static
            Gem::AtomLyIntegration_CommonFeatures.Static