Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request #9981 from OatmealDome/mvk-external-project
MoltenVK: Use an external project instead of a pre-compiled dylib
  • Loading branch information
leoetlino committed Nov 13, 2021
2 parents 3e84279 + 2209dc0 commit 0d4c827
Show file tree
Hide file tree
Showing 6 changed files with 30 additions and 5 deletions.
4 changes: 4 additions & 0 deletions CMakeLists.txt
Expand Up @@ -573,6 +573,10 @@ add_subdirectory(Externals/glslang)

if(ENABLE_VULKAN)
add_definitions(-DHAS_VULKAN)

if(APPLE)
add_subdirectory(Externals/MoltenVK)
endif()
endif()

if(NOT WIN32 OR (NOT (CMAKE_SYSTEM_PROCESSOR STREQUAL "aarch64")))
Expand Down
18 changes: 18 additions & 0 deletions Externals/MoltenVK/CMakeLists.txt
@@ -0,0 +1,18 @@
include(ExternalProject)

ExternalProject_Add(MoltenVK
GIT_REPOSITORY https://github.com/KhronosGroup/MoltenVK.git
GIT_TAG v1.1.5

CONFIGURE_COMMAND <SOURCE_DIR>/fetchDependencies --macos

BUILD_COMMAND make -C <SOURCE_DIR> macos
BUILD_IN_SOURCE ON
BUILD_BYPRODUCTS <SOURCE_DIR>/Package/Release/MoltenVK/dylib/macOS/libMoltenVK.dylib

INSTALL_COMMAND ""

LOG_CONFIGURE ON
LOG_BUILD ON
LOG_OUTPUT_ON_FAILURE ON
)
Binary file removed Externals/MoltenVK/libvulkan.dylib
Binary file not shown.
1 change: 0 additions & 1 deletion Externals/MoltenVK/version.txt

This file was deleted.

8 changes: 6 additions & 2 deletions Source/Core/DolphinQt/CMakeLists.txt
Expand Up @@ -528,8 +528,12 @@ if(APPLE)
endforeach()

# Copy MoltenVK into the bundle
target_sources(dolphin-emu PRIVATE "${CMAKE_SOURCE_DIR}/Externals/MoltenVK/libvulkan.dylib")
set_source_files_properties("${CMAKE_SOURCE_DIR}/Externals/MoltenVK/libvulkan.dylib" PROPERTIES MACOSX_PACKAGE_LOCATION Frameworks)
if(ENABLE_VULKAN)
add_dependencies(dolphin-emu MoltenVK)
ExternalProject_Get_Property(MoltenVK SOURCE_DIR)
target_sources(dolphin-emu PRIVATE "${SOURCE_DIR}/Package/Release/MoltenVK/dylib/macOS/libMoltenVK.dylib")
set_source_files_properties("${SOURCE_DIR}/Package/Release/MoltenVK/dylib/macOS/libMoltenVK.dylib" PROPERTIES MACOSX_PACKAGE_LOCATION Frameworks GENERATED ON)
endif()

# Update library references to make the bundle portable
include(DolphinPostprocessBundle)
Expand Down
4 changes: 2 additions & 2 deletions Source/Core/VideoBackends/Vulkan/VulkanLoader.cpp
Expand Up @@ -43,8 +43,8 @@ static bool OpenVulkanLibrary()
if (libvulkan_env && s_vulkan_module.Open(libvulkan_env))
return true;

// Use the libvulkan.dylib from the application bundle.
std::string filename = File::GetBundleDirectory() + "/Contents/Frameworks/libvulkan.dylib";
// Use the libMoltenVK.dylib from the application bundle.
std::string filename = File::GetBundleDirectory() + "/Contents/Frameworks/libMoltenVK.dylib";
return s_vulkan_module.Open(filename.c_str());
#else
std::string filename = Common::DynamicLibrary::GetVersionedFilename("vulkan", 1);
Expand Down

0 comments on commit 0d4c827

Please sign in to comment.