Skip to content

Commit

Permalink
Simplify library naming
Browse files Browse the repository at this point in the history
  • Loading branch information
JesseTG committed Nov 2, 2023
1 parent 218e091 commit 891c376
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 19 deletions.
9 changes: 2 additions & 7 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,11 +35,6 @@ on:
type: string
required: false
default: "build/dist"
lib-name:
description: The name of the library to build.
type: string
required: false
default: melondsds_libretro
lib-ext:
description: The extension of the library to build.
type: string
Expand Down Expand Up @@ -108,8 +103,8 @@ jobs:
INFO_PATH: "${{ env.DIST_DIR }}/${{ inputs.info-dir }}"
run: |
mkdir -vp "$CORES_PATH" "$INFO_PATH"
cp -f "${{ env.BUILD_DIR }}/src/libretro/${{ inputs.lib-name }}.${{ inputs.lib-ext }}" "$CORES_PATH"
cp -f "${{ github.workspace }}/melondsds_libretro.info" "$INFO_PATH/${{ inputs.lib-name }}.info"
cp -f "${{ env.BUILD_DIR }}/src/libretro/melondsds_libretro.${{ inputs.lib-ext }}" "$CORES_PATH"
cp -f "${{ github.workspace }}/melondsds_libretro.info" "$INFO_PATH/melondsds_libretro.info"
- name: Upload Artifact
uses: actions/upload-artifact@v3
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ jobs:
lib-ext: so
target: android
archive-name: melondsds_libretro-android
lib-name: melondsds_libretro
cmake-args: -DENABLE_OGLRENDERER=OFF -DANDROID_ABI=arm64-v8a -DANDROID_PLATFORM=24 -DCMAKE_TOOLCHAIN_FILE="$ANDROID_NDK/build/cmake/android.toolchain.cmake"
runs-on: ubuntu-latest
# Disabled OpenGL on Android due to https://github.com/JesseTG/melonds-ds/issues/23
Expand All @@ -75,7 +74,6 @@ jobs:
archive-name: melondsds_libretro-ios
info-dir: info
lib-ext: dylib
lib-name: melondsds_libretro_ios
cmake-args: -DENABLE_OGLRENDERER=OFF -DENABLE_JIT=OFF --toolchain ./cmake/toolchain/ios.toolchain.cmake -DPLATFORM=OS64 -DCMAKE_OSX_DEPLOYMENT_TARGET:STRING=13.0 -DDEPLOYMENT_TARGET=14 -DENABLE_STRICT_TRY_COMPILE_INT:BOOL=ON
# Disabled OpenGL on iOS due to https://github.com/JesseTG/melonds-ds/issues/23

Expand Down
11 changes: 1 addition & 10 deletions src/libretro/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -181,16 +181,7 @@ if (ENABLE_JIT)
endif ()

# libretro cores do not start with "lib"
set_target_properties(libretro PROPERTIES PREFIX "")

# Most platforms use the name "<core name>_libretro", but a few have their own conventions
if(CMAKE_SYSTEM_NAME STREQUAL "tvOS")
set_target_properties(libretro PROPERTIES OUTPUT_NAME "melondsds_libretro_tvos")
elseif(IOS)
set_target_properties(libretro PROPERTIES OUTPUT_NAME "melondsds_libretro_ios")
else()
set_target_properties(libretro PROPERTIES OUTPUT_NAME "melondsds_libretro")
endif()
set_target_properties(libretro PROPERTIES PREFIX "" OUTPUT_NAME "melondsds_libretro")

# Some platforms or compilers don't use the expected suffixes for shared libraries
if (APPLE)
Expand Down

0 comments on commit 891c376

Please sign in to comment.