Skip to content

Commit

Permalink
Update mixxx to latest commit on whanake-music branch.
Browse files Browse the repository at this point in the history
Add soft dependencies for Qt 6 (future-proofing).

Also add hard dependency on gtest as compile would fail without it.
  • Loading branch information
icedream committed Sep 24, 2023
1 parent ceeb8bd commit 048bdc4
Show file tree
Hide file tree
Showing 3 changed files with 50 additions and 30 deletions.
54 changes: 25 additions & 29 deletions buildroot-customizations/package/mixxx/0001-Drop-mixxx-test.patch
Original file line number Diff line number Diff line change
@@ -1,20 +1,24 @@
From 5724785c1467223f42448f4558eb522e393abe23 Mon Sep 17 00:00:00 2001
From 386a98e50ea0688069516215d0aac866eb84b56c Mon Sep 17 00:00:00 2001
From: Carl Kittelberger <icedream@icedream.pw>
Date: Tue, 30 May 2023 02:21:21 +0200
Subject: [PATCH] Drop mixxx-test.
Date: Sun, 24 Sep 2023 16:02:59 +0200
Subject: [PATCH 1/2] Drop mixxx-test.

---
CMakeLists.txt | 156 -------------------------------------------------
1 file changed, 156 deletions(-)
CMakeLists.txt | 152 -------------------------------------------------
1 file changed, 152 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 347b0838fa..300d9f79c9 100644
index f59cb3ab0f..75871696fa 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -1681,159 +1681,6 @@ add_subdirectory(
"${CMAKE_CURRENT_BINARY_DIR}/lib/googletest"
)
@@ -1816,155 +1816,6 @@ if(WIN32 AND NOT QT6)
set(APPLOCAL_COMPONENT_DEFINED true)
endif()

-#
-# Tests
-#
-
-add_executable(mixxx-test
- src/test/analyserwaveformtest.cpp
- src/test/analyzersilence_test.cpp
Expand All @@ -30,13 +34,15 @@ index 347b0838fa..300d9f79c9 100644
- src/test/broadcastsettings_test.cpp
- src/test/cache_test.cpp
- src/test/channelhandle_test.cpp
- src/test/chrono_clock_resolution_test.cpp
- src/test/colorconfig_test.cpp
- src/test/colormapperjsproxy_test.cpp
- src/test/colorpalette_test.cpp
- src/test/configobject_test.cpp
- src/test/controller_mapping_validation_test.cpp
- src/test/controllerscriptenginelegacy_test.cpp
- src/test/controlobjecttest.cpp
- src/test/controlobjectaliastest.cpp
- src/test/controlobjectscripttest.cpp
- src/test/coreservicestest.cpp
- src/test/coverartcache_test.cpp
Expand All @@ -55,7 +61,7 @@ index 347b0838fa..300d9f79c9 100644
- src/test/enginebuffertest.cpp
- src/test/engineeffectsdelay_test.cpp
- src/test/enginefilterbiquadtest.cpp
- src/test/enginemastertest.cpp
- src/test/enginemixertest.cpp
- src/test/enginemicrophonetest.cpp
- src/test/enginesynctest.cpp
- src/test/fileinfo_test.cpp
Expand All @@ -64,6 +70,7 @@ index 347b0838fa..300d9f79c9 100644
- src/test/hotcuecontrol_test.cpp
- src/test/imageutils_test.cpp
- src/test/indexrange_test.cpp
- src/test/itunesxmlimportertest.cpp
- src/test/keyutilstest.cpp
- src/test/lcstest.cpp
- src/test/learningutilstest.cpp
Expand Down Expand Up @@ -122,25 +129,14 @@ index 347b0838fa..300d9f79c9 100644
- src/test/wbatterytest.cpp
- src/test/wpushbutton_test.cpp
- src/test/wwidgetstack_test.cpp
- src/util/moc_included_test.cpp
-)
-find_package(GTest CONFIG REQUIRED)
-set_target_properties(mixxx-test PROPERTIES AUTOMOC ON)
-target_link_libraries(mixxx-test PRIVATE mixxx-lib mixxx-gitinfostore gtest gmock)
-
-#
-# Benchmark tests
-#
-
-# Disable testing of google/benchmark
-set(BENCHMARK_ENABLE_TESTING OFF CACHE BOOL "" FORCE)
-target_link_libraries(mixxx-test PRIVATE mixxx-lib mixxx-gitinfostore GTest::gtest GTest::gmock)
-
-# Prevent installation of google/benchmark artifacts
-set(BENCHMARK_ENABLE_INSTALL OFF CACHE BOOL "Disable installation of google/benchmark" FORCE)
-
-add_subdirectory(
- "${CMAKE_CURRENT_SOURCE_DIR}/lib/benchmark"
- "${CMAKE_CURRENT_BINARY_DIR}/lib/benchmark"
-)
-target_link_libraries(mixxx-test PRIVATE benchmark)
-find_package(benchmark)
-target_link_libraries(mixxx-test PRIVATE benchmark::benchmark)
-
-# Test Suite
-include(CTest)
Expand Down Expand Up @@ -171,7 +167,7 @@ index 347b0838fa..300d9f79c9 100644
#
# Resources
#
@@ -1863,8 +1710,6 @@ endif()
@@ -1994,8 +1845,6 @@ endif()

target_sources(mixxx PRIVATE res/mixxx.qrc)
set_target_properties(mixxx PROPERTIES AUTORCC ON)
Expand All @@ -180,7 +176,7 @@ index 347b0838fa..300d9f79c9 100644

if (MIXXX_VERSION_PRERELEASE STREQUAL "")
set(MIXXX_VERSION "${CMAKE_PROJECT_VERSION}")
@@ -2791,7 +2636,6 @@ if(LILV)
@@ -3012,7 +2861,6 @@ if(LILV)
)
target_compile_definitions(mixxx-lib PUBLIC __LILV__)
target_link_libraries(mixxx-lib PRIVATE lilv::lilv)
Expand All @@ -189,5 +185,5 @@ index 347b0838fa..300d9f79c9 100644

# Live Broadcasting (Shoutcast)
--
2.40.1
2.42.0

12 changes: 12 additions & 0 deletions buildroot-customizations/package/mixxx/Config.in
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,9 @@ menuconfig BR2_PACKAGE_MIXXX
select BR2_PACKAGE_RUBBERBAND
select BR2_PACKAGE_TAGLIB

# Dependencies listed in source code itself
select BR2_PACKAGE_GTEST

# Mixxx has shipped protobuf stuff but idk if it works like that in buildroot…?
select BR2_PACKAGE_PROTOBUF
select BR2_PACKAGE_PROTOBUF_C
Expand Down Expand Up @@ -250,8 +253,17 @@ config BR2_PACKAGE_MIXXX_QOPENGL
default BR2_PACKAGE_QT5BASE_OPENGL
depends on BR2_PACKAGE_QT5BASE_OPENGL

config BR2_PACKAGE_MIXXX_QML
bool "Use QML for GUI"
default n
depends on BR2_PACKAGE_QT6

comment "mixxx will use QGLWidget, if you want it to use QOpenGLWindow enable OpenGL support in Qt first"
depends on !BR2_PACKAGE_QT5BASE_OPENGL

comment "mixxx will not use QML as that requires Qt 6"
depends on !BR2_PACKAGE_QT6

endif

comment "mixxx needs chromaprint"
Expand Down
14 changes: 13 additions & 1 deletion buildroot-customizations/package/mixxx/mixxx.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
MIXXX_VERSION = 498395527963f85f7bd959f6a84e4cf7abfb5f67
MIXXX_VERSION = b5c7cca3efc8bbb123a5445fb10ef9606398fe2c
#MIXXX_SOURCE = mixxx-$(MIXXX_VERSION).tar.gz
MIXXX_SITE = https://github.com/whanake-music/mixxx.git
MIXXX_SITE_METHOD = git
Expand All @@ -22,6 +22,7 @@ MIXXX_CONF_OPTS = -DUSE_SYMLINKS=OFF
# 3. CMakeLists.txt shipped with mixxx
MIXXX_DEPENDENCIES = \
chromaprint \
gtest \
hidapi \
lame \
libgl \
Expand Down Expand Up @@ -52,6 +53,17 @@ ifeq ($(BR2_STATIC_DEPS),y)
MIXXX_CONF_OPTS += -DSTATIC_LIBS=ON
endif

ifeq ($(BR2_PACKAGE_QT6),y)
MIXXX_CONF_OPTS += -DQT6=ON
ifeq ($(BR2_PACKAGE_MIXXX_QML),y)
MIXXX_CONF_OPTS += -DQML=ON
else
MIXXX_CONF_OPTS += -DQML=OFF
endif
else
MIXXX_CONF_OPTS += -DQML=OFF -DQT6=OFF
endif

ifeq ($(BR2_PACKAGE_MIXXX_QOPENGL),y)
MIXXX_CONF_OPTS += -DQOPENGL=ON
else
Expand Down

0 comments on commit 048bdc4

Please sign in to comment.