Skip to content

Commit

Permalink
Exclude openssl from native installation, rather than from build.
Browse files Browse the repository at this point in the history
  • Loading branch information
kring committed Aug 16, 2024
1 parent fc4ff10 commit 1dd7194
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 9 deletions.
4 changes: 0 additions & 4 deletions Source/CesiumEditor/CesiumEditor.Build.cs
Original file line number Diff line number Diff line change
Expand Up @@ -73,10 +73,6 @@ public CesiumEditor(ReadOnlyTargetRules Target) : base(Target)

string[] allLibs = Directory.GetFiles(libPath, libSearchPattern);

// Remove libssl and libcrypto because these come from Unreal Engine itself
// TODO: Exclude these from the cesium-native side instead of excluding them here.
allLibs = allLibs.Where(l => !l.Contains("libssl") && !l.Contains("libcrypto")).ToArray();

PublicAdditionalLibraries.AddRange(allLibs);

PublicDependencyModuleNames.AddRange(
Expand Down
4 changes: 0 additions & 4 deletions Source/CesiumRuntime/CesiumRuntime.Build.cs
Original file line number Diff line number Diff line change
Expand Up @@ -72,10 +72,6 @@ public CesiumRuntime(ReadOnlyTargetRules Target) : base(Target)

string[] allLibs = Directory.GetFiles(libPath, libSearchPattern);

// Remove libssl and libcrypto because these come from Unreal Engine itself
// TODO: Exclude these from the cesium-native side instead of excluding them here.
allLibs = allLibs.Where(l => !l.Contains("libssl") && !l.Contains("libcrypto")).ToArray();

PublicAdditionalLibraries.AddRange(allLibs);

PublicDependencyModuleNames.AddRange(
Expand Down
4 changes: 4 additions & 0 deletions extern/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,10 @@ if (NOT VCPKG_TRIPLET)
endif()
endif()

# Our OpenSSL is part of Unreal Engine, so need to install it.
set(CESIUM_EXCLUDE_INSTALL_HEADERS openssl)
set(CESIUM_EXCLUDE_INSTALL_STATIC_LIBS openssl)

add_subdirectory(cesium-native)

# cesium-native doesn't require this header to be public, but Cesium for Unreal wants to use it.
Expand Down

0 comments on commit 1dd7194

Please sign in to comment.