Skip to content

Commit 8ce93b6

Browse files
author
Rye
committed
Fix SLPlugin bundle needlessly linking in SDL2
1 parent 39c2b06 commit 8ce93b6

File tree

5 files changed

+7
-12
lines changed

5 files changed

+7
-12
lines changed

indra/cmake/PluginAPI.cmake

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,4 +13,6 @@ if (WINDOWS)
1313
)
1414
endif (WINDOWS)
1515

16+
target_include_directories( ll::pluginlibraries INTERFACE ${CMAKE_SOURCE_DIR}/llimage ${CMAKE_SOURCE_DIR}/llrender)
17+
1618

indra/llplugin/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,6 @@ list(APPEND llplugin_SOURCE_FILES ${llplugin_HEADER_FILES})
3535

3636
add_library (llplugin ${llplugin_SOURCE_FILES})
3737
target_include_directories( llplugin INTERFACE ${CMAKE_CURRENT_SOURCE_DIR})
38-
target_link_libraries( llplugin llcommon llmath llrender llmessage )
38+
target_link_libraries( llplugin llcommon llmath llmessage llxml)
3939
add_subdirectory(slplugin)
4040

indra/llplugin/llpluginclassmedia.h

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929
#ifndef LL_LLPLUGINCLASSMEDIA_H
3030
#define LL_LLPLUGINCLASSMEDIA_H
3131

32-
#include "llgltypes.h"
3332
#include "llpluginprocessparent.h"
3433
#include "llrect.h"
3534
#include "llpluginclassmediaowner.h"
@@ -355,9 +354,9 @@ class LLPluginClassMedia : public LLPluginProcessParentOwner
355354

356355
bool mTextureParamsReceived; // the mRequestedTexture* fields are only valid when this is true
357356
S32 mRequestedTextureDepth;
358-
LLGLenum mRequestedTextureInternalFormat;
359-
LLGLenum mRequestedTextureFormat;
360-
LLGLenum mRequestedTextureType;
357+
U32 mRequestedTextureInternalFormat;
358+
U32 mRequestedTextureFormat;
359+
U32 mRequestedTextureType;
361360
bool mRequestedTextureSwapBytes;
362361
bool mRequestedTextureCoordsOpenGL;
363362

indra/media_plugins/base/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,5 +27,5 @@ add_library(media_plugin_base
2727
${media_plugin_base_SOURCE_FILES}
2828
)
2929

30-
target_link_libraries( media_plugin_base llplugin )
30+
target_link_libraries( media_plugin_base llplugin ll::pluginlibraries)
3131
target_include_directories( media_plugin_base INTERFACE ${CMAKE_CURRENT_SOURCE_DIR})

indra/newview/viewer_manifest.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1036,12 +1036,6 @@ def path_optional(src, dst):
10361036
):
10371037
dylibs += path_optional(os.path.join(relpkgdir, libfile), libfile)
10381038

1039-
# SDL2
1040-
for libfile in (
1041-
'libSDL2-2.0.dylib',
1042-
):
1043-
dylibs += path_optional(os.path.join(relpkgdir, libfile), libfile)
1044-
10451039
# our apps
10461040
executable_path = {}
10471041
embedded_apps = [ (os.path.join("llplugin", "slplugin"), "SLPlugin.app") ]

0 commit comments

Comments
 (0)