Skip to content

Commit

Permalink
Merge remote-tracking branch 'mpg123/master' into master-with-github-ci
Browse files Browse the repository at this point in the history
  • Loading branch information
mpg123 GitHub bot committed May 2, 2024
2 parents 1ca9d3f + a6dc858 commit 4e2e301
Show file tree
Hide file tree
Showing 4 changed files with 32 additions and 7 deletions.
20 changes: 20 additions & 0 deletions .github/workflows/CMake-macOS.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
name: CMake macOS

on: push

jobs:
build:
strategy:
matrix:
os: [macos-latest, macos-13]
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v4
- name: Install ninja & yasm
run: brew install ninja yasm
- name: Configure (CMake)
run: |
cmake -S ports/cmake -B build -GNinja
- name: Build (CMake)
run: |
cmake --build build
5 changes: 5 additions & 0 deletions NEWS
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
1.32.7
------
- ports/cmake: Work around bug in CMake that does not detect FPU on
Apple ARM CPUs (github PR 14).

1.32.6
------
- build: Detect forced 64 bit offsets on a dual-mode system that used
Expand Down
8 changes: 7 additions & 1 deletion ports/cmake/src/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,13 @@ if(NO_MESSAGES)
set(NO_ERETURN ON)
endif()

if(WIN32)
include(../cmake/CheckCPUArch.cmake)
check_cpu_arch_x86(ARCH_IS_X86)
check_cpu_arch_x64(ARCH_IS_X64)
check_cpu_arch_arm32(ARCH_IS_ARM32)
check_cpu_arch_arm64(ARCH_IS_ARM64)

if(WIN32 OR (ARCH_IS_ARM64 AND APPLE))
set(HAVE_FPU 1)
else()
cmake_host_system_information(RESULT HAVE_FPU QUERY HAS_FPU)
Expand Down
6 changes: 0 additions & 6 deletions ports/cmake/src/libmpg123/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,6 @@ endif()

include_directories("${CMAKE_CURRENT_BINARY_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}/../../../../src/libmpg123/")

include(../../cmake/CheckCPUArch.cmake)

if(HAVE_STDLIB_H)
set(INCLUDE_STDLIB_H "#include <stdlib.h>")
else()
Expand All @@ -21,10 +19,6 @@ else()
endif()


check_cpu_arch_x86(ARCH_IS_X86)
check_cpu_arch_x64(ARCH_IS_X64)
check_cpu_arch_arm32(ARCH_IS_ARM32)
check_cpu_arch_arm64(ARCH_IS_ARM64)

# PPC with AltiVec is missing. But probably obsolete enough for CMake users.

Expand Down

0 comments on commit 4e2e301

Please sign in to comment.