commit 4aa21c418e7bf56e56c66f85437b9925efc4b742
Author: Uwe Klotz <uwe.klotz(a)gmail.com>
Date: Sat Apr 22 15:53:39 2023 +0200
Switch back to GCC 13
aarch64_asm_operand_widths.patch | 16 +++-------------
fidlib-fno-finite-math.patch | 20 ++++++++++++++++++++
mixxx.spec | 22 ++++++++--------------
3 files changed, 31 insertions(+), 27 deletions(-)
---
diff --git a/aarch64_asm_operand_widths.patch b/aarch64_asm_operand_widths.patch
index fdd523d..bd00507 100644
--- a/aarch64_asm_operand_widths.patch
+++ b/aarch64_asm_operand_widths.patch
@@ -1,13 +1,3 @@
-From 38a145c6ba4022c70f36290c7f52d5cb91826a95 Mon Sep 17 00:00:00 2001
-From: Uwe Klotz <uwe.klotz(a)gmail.com>
-Date: Mon, 17 Apr 2023 22:41:32 +0200
-Subject: [PATCH] aarch64: Fix -Wasm-operand-widths
-
----
- src/soundio/sounddevicenetwork.cpp | 2 +-
- src/soundio/sounddeviceportaudio.cpp | 2 +-
- 2 files changed, 2 insertions(+), 2 deletions(-)
-
diff --git a/src/soundio/sounddevicenetwork.cpp b/src/soundio/sounddevicenetwork.cpp
index aff65b9dc0..8da832d6a9 100644
--- a/src/soundio/sounddevicenetwork.cpp
@@ -20,7 +10,7 @@ index aff65b9dc0..8da832d6a9 100644
+ int64_t savedFPCR;
asm volatile("mrs %[savedFPCR], FPCR"
: [ savedFPCR ] "=r"(savedFPCR));
-
+
diff --git a/src/soundio/sounddeviceportaudio.cpp b/src/soundio/sounddeviceportaudio.cpp
index 681d0942cc..97a4429420 100644
--- a/src/soundio/sounddeviceportaudio.cpp
@@ -33,7 +23,7 @@ index 681d0942cc..97a4429420 100644
+ int64_t savedFPCR;
asm volatile("mrs %[savedFPCR], FPCR"
: [ savedFPCR ] "=r"(savedFPCR));
-
---
+
+--
2.40.0
diff --git a/fidlib-fno-finite-math.patch b/fidlib-fno-finite-math.patch
new file mode 100644
index 0000000..1e311ef
--- /dev/null
+++ b/fidlib-fno-finite-math.patch
@@ -0,0 +1,20 @@
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index 9a3c93f044..18f30caf62 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -1852,10 +1852,13 @@ add_library(fidlib STATIC EXCLUDE_FROM_ALL lib/fidlib/fidlib.c)
+ if(MSVC)
+ target_compile_definitions(fidlib PRIVATE T_MSVC)
+ target_compile_definitions(fidlib PRIVATE _USE_MATH_DEFINES)
++ target_compile_options(fidlib PRIVATE /W3)
+ elseif(MINGW)
+ target_compile_definitions(fidlib PRIVATE T_MINGW)
++ target_compile_options(fidlib PRIVATE -fno-finite-math-only)
+ else()
+ target_compile_definitions(fidlib PRIVATE T_LINUX)
++ target_compile_options(fidlib PRIVATE -fno-finite-math-only)
+ endif()
+ target_include_directories(mixxx-lib SYSTEM PUBLIC lib/fidlib)
+ target_link_libraries(mixxx-lib PRIVATE fidlib)
+--
+2.40.0
diff --git a/mixxx.spec b/mixxx.spec
index 2702f99..42982e8 100644
--- a/mixxx.spec
+++ b/mixxx.spec
@@ -25,7 +25,7 @@
Name: mixxx
Version: 2.3.4
-Release: 3%{?extraver:.%{extraver}}%{?snapinfo:.%{snapinfo}}%{?dist}
+Release: 4%{?extraver:.%{extraver}}%{?snapinfo:.%{snapinfo}}%{?dist}
Summary: Mixxx is open source software for DJ'ing
License: GPLv2+
URL:
http://www.mixxx.org
@@ -35,10 +35,7 @@ Source0:
https://github.com/mixxxdj/%{name}/archive/%{sources}/%{name}-%{
Source1:
https://github.com/mixxxdj/libkeyfinder/archive/refs/tags/v%{libkeyfinder...
Patch0: disable_werror_in_tests.patch
Patch1: aarch64_asm_operand_widths.patch
-
-# TODO: Switch back from "clang" to "gcc" after issues have been
fixed?
-# See also: <
https://github.com/mixxxdj/mixxx/issues/11483>
-%global toolchain clang
+Patch2: fidlib-fno-finite-math.patch
# Build Tools
BuildRequires: desktop-file-utils
@@ -46,7 +43,7 @@ BuildRequires: appstream
BuildRequires: protobuf-compiler
BuildRequires: cmake
BuildRequires: ccache
-BuildRequires: clang
+BuildRequires: gcc-c++
BuildRequires: ninja-build
# Build Requirements
@@ -125,9 +122,6 @@ cp %{SOURCE1} %{__cmake_builddir}/downloads
%cmake \
-GNinja \
- -DCMAKE_C_COMPILER=clang \
- -DCMAKE_CXX_COMPILER=clang++ \
- -DCMAKE_LINKER=clang++ \
-DCMAKE_BUILD_TYPE=RelWithDebInfo \
-DOPTIMIZE=portable \
-DINSTALL_USER_UDEV_RULES=ON \
@@ -170,22 +164,19 @@ rm -rf \
%check
-# TODO: Re-enable Denormal test after Mixxx v2.3.5 has been released.
-# <
https://github.com/mixxxdj/mixxx/issues/11484>
-
# TODO: Enable EngineBufferE2ETest after spurious failures for
# x86_64 when run on AMD EPYC have been resolved. Varying tests
# are failing sometimes.
%ifarch x86_64
%global ctest_timeout_secs 180
- %global ctest_exclude_regex EngineBufferE2ETest|Denormal
+ %global ctest_exclude_regex EngineBufferE2ETest
%endif
# TODO: Enable ControllerEngine NaN tests on ARM after the cause for
# the failing tests has been found and fixed.
%ifarch %{arm32} %{arm64}
%global ctest_timeout_secs 300
- %global ctest_exclude_regex setValue_IgnoresNaN|setParameter_NaN|Denormal
+ %global ctest_exclude_regex setValue_IgnoresNaN|setParameter_NaN
%endif
%ifarch %{power64}
@@ -222,6 +213,9 @@ appstreamcli \
%{_udevrulesdir}/69-%{name}-usb-uaccess.rules
%changelog
+* Sat Apr 22 2023 Uwe Klotz <uklotz(a)gmail.com> - 2.3.4-4
+- Switch back to GCC 13
+
* Mon Apr 17 2023 Uwe Klotz <uklotz(a)gmail.com> - 2.3.4-3
- Switch from GCC 13 to Clang 16