Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] mixxx: update to 2.3.3.
@ 2022-08-02  0:34 abenson
  2022-08-02  2:12 ` [PR PATCH] [Updated] " abenson
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: abenson @ 2022-08-02  0:34 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 601 bytes --]

There is a new pull request by abenson against master on the void-packages repository

https://github.com/abenson/void-packages mixxx_2.3.3
https://github.com/void-linux/void-packages/pull/38416

mixxx: update to 2.3.3.

#### Testing the changes
- I tested the changes in this PR: **briefly**

#### Local build testing
- I built this PR locally for my native architecture, x86_64
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - x86_64-musl
  - aarch64 (cross)


A patch file from https://github.com/void-linux/void-packages/pull/38416.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mixxx_2.3.3-38416.patch --]
[-- Type: text/x-diff, Size: 8075 bytes --]

From e388b62680ec0881b875cd03724b988a171df35b Mon Sep 17 00:00:00 2001
From: Andrew Benson <abenson+void@gmail.com>
Date: Mon, 1 Aug 2022 19:15:45 -0500
Subject: [PATCH] mixxx: update to 2.3.3.

---
 srcpkgs/mixxx/patches/fix-musl.patch | 101 +++++++++++++++++++++++++++
 srcpkgs/mixxx/template               |  49 +++----------
 2 files changed, 111 insertions(+), 39 deletions(-)
 create mode 100644 srcpkgs/mixxx/patches/fix-musl.patch

diff --git a/srcpkgs/mixxx/patches/fix-musl.patch b/srcpkgs/mixxx/patches/fix-musl.patch
new file mode 100644
index 000000000000..ba93de072180
--- /dev/null
+++ b/srcpkgs/mixxx/patches/fix-musl.patch
@@ -0,0 +1,101 @@
+--- a/src/sources/soundsourcem4a.h
++++ b/src/sources/soundsourcem4a.h
+@@ -58,10 +58,10 @@
+     MP4Duration m_framesPerSampleBlock;
+     MP4SampleId m_maxSampleBlockId;
+ 
+-    u_int8_t* m_pMP4ESConfigBuffer{};
+-    u_int32_t m_sizeofMP4ESConfigBuffer;
++    uint8_t* m_pMP4ESConfigBuffer{};
++    uint32_t m_sizeofMP4ESConfigBuffer;
+ 
+-    typedef std::vector<u_int8_t> InputBuffer;
++    typedef std::vector<uint8_t> InputBuffer;
+     InputBuffer m_inputBuffer;
+     InputBuffer::size_type m_inputBufferLength;
+     InputBuffer::size_type m_inputBufferOffset;
+--- a/src/sources/soundsourcem4a.cpp
++++ b/src/sources/soundsourcem4a.cpp
+@@ -36,7 +36,7 @@
+ constexpr SINT kNumberOfPrefetchFrames = 2112;
+ 
+ // The TrackId is a 1-based index of the tracks in an MP4 file
+-constexpr u_int32_t kMinTrackId = 1;
++constexpr uint32_t kMinTrackId = 1;
+ 
+ // http://www.iis.fraunhofer.de/content/dam/iis/de/doc/ame/wp/FraunhoferIIS_Application-Bulletin_AAC-Transport-Formats.pdf
+ // Footnote 13: "The usual frame length for AAC-LC is 1024 samples, but a 960 sample version
+@@ -52,9 +52,9 @@
+ //   https://github.com/sannies/mp4parser/blob/master/isoparser/src/main/java/org/mp4parser/boxes/iso14496/part1/objectdescriptors/DecoderConfigDescriptor.java
+ //   http://mutagen-specs.readthedocs.io/en/latest/mp4/
+ //   http://perso.telecom-paristech.fr/~dufourd/mpeg-4/tools.html
+-constexpr u_int32_t kMaxSampleBlockInputSizeLimit = (u_int32_t(1) << 24) - 1;
++constexpr uint32_t kMaxSampleBlockInputSizeLimit = (uint32_t(1) << 24) - 1;
+ 
+-inline u_int32_t getMaxTrackId(MP4FileHandle hFile) {
++inline uint32_t getMaxTrackId(MP4FileHandle hFile) {
+     // The maximum TrackId equals the number of all tracks
+     // in an MP4 file. We pass nullptr and 0 as arguments
+     // to avoid any type/subtype filtering at this point!
+@@ -76,8 +76,8 @@
+ // Searches for the first audio track in the MP4 file that
+ // suits our needs.
+ MP4TrackId findFirstAudioTrackId(MP4FileHandle hFile, const QString& fileName) {
+-    const u_int32_t maxTrackId = getMaxTrackId(hFile);
+-    for (u_int32_t trackId = kMinTrackId; trackId <= maxTrackId; ++trackId) {
++    const uint32_t maxTrackId = getMaxTrackId(hFile);
++    for (uint32_t trackId = kMinTrackId; trackId <= maxTrackId; ++trackId) {
+         const char* trackType = MP4GetTrackType(hFile, trackId);
+         if (!isValidTrackType(trackType)) {
+             kLogger.warning() << "Unsupported track type"
+@@ -102,10 +102,10 @@
+                               << fileName;
+             continue;
+         }
+-        const u_int8_t audioType = MP4GetTrackEsdsObjectTypeId(hFile, trackId);
++        const uint8_t audioType = MP4GetTrackEsdsObjectTypeId(hFile, trackId);
+         if (MP4_IS_AAC_AUDIO_TYPE(audioType)) {
+             if (MP4_MPEG4_AUDIO_TYPE == audioType) {
+-                const u_int8_t mpeg4AudioType =
++                const uint8_t mpeg4AudioType =
+                         MP4GetTrackAudioMpeg4Type(hFile, trackId);
+                 if (MP4_IS_MPEG4_AAC_AUDIO_TYPE(mpeg4AudioType)) {
+                     return trackId;
+@@ -151,7 +151,7 @@
+ constexpr size_t kMinADTSHeaderLength = 7;
+ 
+ size_t getADTSHeaderLength(
+-        const u_int8_t* pInputBuffer,
++        const uint8_t* pInputBuffer,
+         size_t sizeofInputBuffer) {
+     // The size of the raw data block must be strictly greater than
+     // the size of only ADTS header alone, i.e. additional sample
+@@ -178,7 +178,7 @@
+ }
+ 
+ inline bool startsWithADTSHeader(
+-        const u_int8_t* pInputBuffer,
++        const uint8_t* pInputBuffer,
+         size_t sizeofInputBuffer) {
+     return 0 < getADTSHeaderLength(pInputBuffer, sizeofInputBuffer);
+ }
+@@ -282,7 +282,7 @@
+ 
+     // Determine the maximum input size (in bytes) of a
+     // sample block for the selected track.
+-    const u_int32_t maxSampleBlockInputSize = MP4GetTrackMaxSampleSize(m_hFile,
++    const uint32_t maxSampleBlockInputSize = MP4GetTrackMaxSampleSize(m_hFile,
+             m_trackId);
+     if (maxSampleBlockInputSize == 0) {
+         kLogger.warning() << "Failed to read MP4 DecoderConfigDescriptor.bufferSizeDB:"
+@@ -627,8 +627,8 @@
+                 // Fill input buffer from file
+                 if (isValidSampleBlockId(m_curSampleBlockId)) {
+                     // Read data for next sample block into input buffer
+-                    u_int8_t* pInputBuffer = m_inputBuffer.data();
+-                    u_int32_t inputBufferLength = m_inputBuffer.size(); // in/out parameter
++                    uint8_t* pInputBuffer = m_inputBuffer.data();
++                    uint32_t inputBufferLength = m_inputBuffer.size(); // in/out parameter
+                     if (!MP4ReadSample(m_hFile,
+                                 m_trackId,
+                                 m_curSampleBlockId,
diff --git a/srcpkgs/mixxx/template b/srcpkgs/mixxx/template
index e5912b482fa1..232b7050c709 100644
--- a/srcpkgs/mixxx/template
+++ b/srcpkgs/mixxx/template
@@ -1,51 +1,22 @@
 # Template file for 'mixxx'
 pkgname=mixxx
-version=2.2.4
-revision=13
-wrksrc="mixxx-release-${version}"
-build_style=scons
-hostmakedepends="pkg-config protobuf"
+version=2.3.3
+revision=1
+build_style=cmake
+configure_args="-DCMAKE_BUILD_TYPE=Release"
+hostmakedepends="pkg-config protobuf qt5-devel"
 makedepends="chromaprint-devel faad2-devel ffmpeg-devel fftw-devel glu-devel
  libid3tag-devel libmad-devel libmp4v2-devel libshout-devel libusb-devel
  opusfile-devel portaudio-devel portmidi-devel protobuf-devel qt5-script-devel
  qt5-svg-devel qt5-xmlpatterns-devel rubberband-devel taglib-devel upower-devel
  vamp-plugin-sdk-devel lv2 lilv-devel qt5-x11extras-devel hidapi-devel
- libtheora-devel speex-devel soundtouch-devel"
+ libtheora-devel speex-devel soundtouch-devel lame-devel qt5-plugin-mysql
+ qt5-plugin-odbc qt5-plugin-pgsql qt5-plugin-sqlite qt5-plugin-tds
+ qtkeychain-qt5-devel"
 depends="qt5-plugin-sqlite"
 short_desc="Open source digital DJing software that allows mixing music"
 maintainer="Enno Boland <gottox@voidlinux.org>"
 license="GPL-2.0-or-later"
 homepage="https://www.mixxx.org"
-distfiles="https://github.com/mixxxdj/mixxx/archive/release-${version}.tar.gz"
-checksum=9372b43d5ec882845b4fe2350ef50dabb3f1e0cc029f182b0ed8aa4f4f3b2afa
-_scons_args="shoutcast=1 faad=1 tuned=0 prefix=/usr qt5=1 qtdir=/usr/include/qt5"
-
-nocross=https://build.voidlinux.org/builders/armv7l_builder/builds/7882/steps/shell_3/logs/stdio
-
-do_build() {
-	case "$XBPS_TARGET_MACHINE" in
-		*-musl) find plugins -type f -exec sed -i "{}" \
-			-e "s;u_int8_t;uint8_t;g" -e "s;u_int32_t;uint32_t;g" \;
-			;;
-	esac
-	if [ "${XBPS_TARGET_MACHINE%-musl}" = "i686" ]; then
-		export CFLAGS+=" -msse"
-		export CXXFLAGS+=" -msse"
-	fi
-	sed -i \
-		-e 's/branch_name = .*/branch_name = "mixxx"/' \
-		-e 's/vcs_revision = .*/vcs_revision = "'$version'"/' \
-		src/SConscript
-
-	sed -i -e 's/vamp-plugin-sdk/vamp-sdk/g' build/features.py vamp-plugins/SConscript
-
-	MAKEFLAGS="${makejobs}" scons ${makejobs} ${_scons_args}
-}
-
-do_install() {
-	if [ "$XBPS_TARGET_MACHINE" = "i686" ]; then
-		export CFLAGS+=" -msse"
-		export CXXFLAGS+=" -msse"
-	fi
-	scons ${_scons_args} install_root=${DESTDIR}/usr install
-}
+distfiles="https://github.com/mixxxdj/mixxx/archive/${version}.tar.gz"
+checksum=8e3a5a545175982336bb07c81a3839897a007c43689b93641242db662f6beb9c

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Updated] mixxx: update to 2.3.3.
  2022-08-02  0:34 [PR PATCH] mixxx: update to 2.3.3 abenson
@ 2022-08-02  2:12 ` abenson
  2022-08-02 13:26 ` prez
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: abenson @ 2022-08-02  2:12 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 606 bytes --]

There is an updated pull request by abenson against master on the void-packages repository

https://github.com/abenson/void-packages mixxx_2.3.3
https://github.com/void-linux/void-packages/pull/38416

mixxx: update to 2.3.3.

#### Testing the changes
- I tested the changes in this PR: **briefly**

#### Local build testing
- I built this PR locally for my native architecture, x86_64
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - x86_64-musl
  - aarch64 (cross)


A patch file from https://github.com/void-linux/void-packages/pull/38416.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mixxx_2.3.3-38416.patch --]
[-- Type: text/x-diff, Size: 8092 bytes --]

From f500aed64635aa8096f8d080e5518a9cbdc3d662 Mon Sep 17 00:00:00 2001
From: Andrew Benson <abenson+void@gmail.com>
Date: Mon, 1 Aug 2022 19:15:45 -0500
Subject: [PATCH] mixxx: update to 2.3.3.

---
 srcpkgs/mixxx/patches/fix-musl.patch | 101 +++++++++++++++++++++++++++
 srcpkgs/mixxx/template               |  49 +++----------
 2 files changed, 111 insertions(+), 39 deletions(-)
 create mode 100644 srcpkgs/mixxx/patches/fix-musl.patch

diff --git a/srcpkgs/mixxx/patches/fix-musl.patch b/srcpkgs/mixxx/patches/fix-musl.patch
new file mode 100644
index 000000000000..ba93de072180
--- /dev/null
+++ b/srcpkgs/mixxx/patches/fix-musl.patch
@@ -0,0 +1,101 @@
+--- a/src/sources/soundsourcem4a.h
++++ b/src/sources/soundsourcem4a.h
+@@ -58,10 +58,10 @@
+     MP4Duration m_framesPerSampleBlock;
+     MP4SampleId m_maxSampleBlockId;
+ 
+-    u_int8_t* m_pMP4ESConfigBuffer{};
+-    u_int32_t m_sizeofMP4ESConfigBuffer;
++    uint8_t* m_pMP4ESConfigBuffer{};
++    uint32_t m_sizeofMP4ESConfigBuffer;
+ 
+-    typedef std::vector<u_int8_t> InputBuffer;
++    typedef std::vector<uint8_t> InputBuffer;
+     InputBuffer m_inputBuffer;
+     InputBuffer::size_type m_inputBufferLength;
+     InputBuffer::size_type m_inputBufferOffset;
+--- a/src/sources/soundsourcem4a.cpp
++++ b/src/sources/soundsourcem4a.cpp
+@@ -36,7 +36,7 @@
+ constexpr SINT kNumberOfPrefetchFrames = 2112;
+ 
+ // The TrackId is a 1-based index of the tracks in an MP4 file
+-constexpr u_int32_t kMinTrackId = 1;
++constexpr uint32_t kMinTrackId = 1;
+ 
+ // http://www.iis.fraunhofer.de/content/dam/iis/de/doc/ame/wp/FraunhoferIIS_Application-Bulletin_AAC-Transport-Formats.pdf
+ // Footnote 13: "The usual frame length for AAC-LC is 1024 samples, but a 960 sample version
+@@ -52,9 +52,9 @@
+ //   https://github.com/sannies/mp4parser/blob/master/isoparser/src/main/java/org/mp4parser/boxes/iso14496/part1/objectdescriptors/DecoderConfigDescriptor.java
+ //   http://mutagen-specs.readthedocs.io/en/latest/mp4/
+ //   http://perso.telecom-paristech.fr/~dufourd/mpeg-4/tools.html
+-constexpr u_int32_t kMaxSampleBlockInputSizeLimit = (u_int32_t(1) << 24) - 1;
++constexpr uint32_t kMaxSampleBlockInputSizeLimit = (uint32_t(1) << 24) - 1;
+ 
+-inline u_int32_t getMaxTrackId(MP4FileHandle hFile) {
++inline uint32_t getMaxTrackId(MP4FileHandle hFile) {
+     // The maximum TrackId equals the number of all tracks
+     // in an MP4 file. We pass nullptr and 0 as arguments
+     // to avoid any type/subtype filtering at this point!
+@@ -76,8 +76,8 @@
+ // Searches for the first audio track in the MP4 file that
+ // suits our needs.
+ MP4TrackId findFirstAudioTrackId(MP4FileHandle hFile, const QString& fileName) {
+-    const u_int32_t maxTrackId = getMaxTrackId(hFile);
+-    for (u_int32_t trackId = kMinTrackId; trackId <= maxTrackId; ++trackId) {
++    const uint32_t maxTrackId = getMaxTrackId(hFile);
++    for (uint32_t trackId = kMinTrackId; trackId <= maxTrackId; ++trackId) {
+         const char* trackType = MP4GetTrackType(hFile, trackId);
+         if (!isValidTrackType(trackType)) {
+             kLogger.warning() << "Unsupported track type"
+@@ -102,10 +102,10 @@
+                               << fileName;
+             continue;
+         }
+-        const u_int8_t audioType = MP4GetTrackEsdsObjectTypeId(hFile, trackId);
++        const uint8_t audioType = MP4GetTrackEsdsObjectTypeId(hFile, trackId);
+         if (MP4_IS_AAC_AUDIO_TYPE(audioType)) {
+             if (MP4_MPEG4_AUDIO_TYPE == audioType) {
+-                const u_int8_t mpeg4AudioType =
++                const uint8_t mpeg4AudioType =
+                         MP4GetTrackAudioMpeg4Type(hFile, trackId);
+                 if (MP4_IS_MPEG4_AAC_AUDIO_TYPE(mpeg4AudioType)) {
+                     return trackId;
+@@ -151,7 +151,7 @@
+ constexpr size_t kMinADTSHeaderLength = 7;
+ 
+ size_t getADTSHeaderLength(
+-        const u_int8_t* pInputBuffer,
++        const uint8_t* pInputBuffer,
+         size_t sizeofInputBuffer) {
+     // The size of the raw data block must be strictly greater than
+     // the size of only ADTS header alone, i.e. additional sample
+@@ -178,7 +178,7 @@
+ }
+ 
+ inline bool startsWithADTSHeader(
+-        const u_int8_t* pInputBuffer,
++        const uint8_t* pInputBuffer,
+         size_t sizeofInputBuffer) {
+     return 0 < getADTSHeaderLength(pInputBuffer, sizeofInputBuffer);
+ }
+@@ -282,7 +282,7 @@
+ 
+     // Determine the maximum input size (in bytes) of a
+     // sample block for the selected track.
+-    const u_int32_t maxSampleBlockInputSize = MP4GetTrackMaxSampleSize(m_hFile,
++    const uint32_t maxSampleBlockInputSize = MP4GetTrackMaxSampleSize(m_hFile,
+             m_trackId);
+     if (maxSampleBlockInputSize == 0) {
+         kLogger.warning() << "Failed to read MP4 DecoderConfigDescriptor.bufferSizeDB:"
+@@ -627,8 +627,8 @@
+                 // Fill input buffer from file
+                 if (isValidSampleBlockId(m_curSampleBlockId)) {
+                     // Read data for next sample block into input buffer
+-                    u_int8_t* pInputBuffer = m_inputBuffer.data();
+-                    u_int32_t inputBufferLength = m_inputBuffer.size(); // in/out parameter
++                    uint8_t* pInputBuffer = m_inputBuffer.data();
++                    uint32_t inputBufferLength = m_inputBuffer.size(); // in/out parameter
+                     if (!MP4ReadSample(m_hFile,
+                                 m_trackId,
+                                 m_curSampleBlockId,
diff --git a/srcpkgs/mixxx/template b/srcpkgs/mixxx/template
index e5912b482fa1..49b879a4526a 100644
--- a/srcpkgs/mixxx/template
+++ b/srcpkgs/mixxx/template
@@ -1,51 +1,22 @@
 # Template file for 'mixxx'
 pkgname=mixxx
-version=2.2.4
-revision=13
-wrksrc="mixxx-release-${version}"
-build_style=scons
-hostmakedepends="pkg-config protobuf"
+version=2.3.3
+revision=1
+build_style=cmake
+configure_args="-DCMAKE_BUILD_TYPE=Release"
+hostmakedepends="pkg-config protobuf qt5-devel"
 makedepends="chromaprint-devel faad2-devel ffmpeg-devel fftw-devel glu-devel
  libid3tag-devel libmad-devel libmp4v2-devel libshout-devel libusb-devel
  opusfile-devel portaudio-devel portmidi-devel protobuf-devel qt5-script-devel
  qt5-svg-devel qt5-xmlpatterns-devel rubberband-devel taglib-devel upower-devel
  vamp-plugin-sdk-devel lv2 lilv-devel qt5-x11extras-devel hidapi-devel
- libtheora-devel speex-devel soundtouch-devel"
+ libtheora-devel speex-devel soundtouch-devel lame-devel qt5-plugin-mysql
+ qt5-plugin-odbc qt5-plugin-pgsql qt5-plugin-sqlite qt5-plugin-tds
+ qtkeychain-qt5-devel libebur128-devel"
 depends="qt5-plugin-sqlite"
 short_desc="Open source digital DJing software that allows mixing music"
 maintainer="Enno Boland <gottox@voidlinux.org>"
 license="GPL-2.0-or-later"
 homepage="https://www.mixxx.org"
-distfiles="https://github.com/mixxxdj/mixxx/archive/release-${version}.tar.gz"
-checksum=9372b43d5ec882845b4fe2350ef50dabb3f1e0cc029f182b0ed8aa4f4f3b2afa
-_scons_args="shoutcast=1 faad=1 tuned=0 prefix=/usr qt5=1 qtdir=/usr/include/qt5"
-
-nocross=https://build.voidlinux.org/builders/armv7l_builder/builds/7882/steps/shell_3/logs/stdio
-
-do_build() {
-	case "$XBPS_TARGET_MACHINE" in
-		*-musl) find plugins -type f -exec sed -i "{}" \
-			-e "s;u_int8_t;uint8_t;g" -e "s;u_int32_t;uint32_t;g" \;
-			;;
-	esac
-	if [ "${XBPS_TARGET_MACHINE%-musl}" = "i686" ]; then
-		export CFLAGS+=" -msse"
-		export CXXFLAGS+=" -msse"
-	fi
-	sed -i \
-		-e 's/branch_name = .*/branch_name = "mixxx"/' \
-		-e 's/vcs_revision = .*/vcs_revision = "'$version'"/' \
-		src/SConscript
-
-	sed -i -e 's/vamp-plugin-sdk/vamp-sdk/g' build/features.py vamp-plugins/SConscript
-
-	MAKEFLAGS="${makejobs}" scons ${makejobs} ${_scons_args}
-}
-
-do_install() {
-	if [ "$XBPS_TARGET_MACHINE" = "i686" ]; then
-		export CFLAGS+=" -msse"
-		export CXXFLAGS+=" -msse"
-	fi
-	scons ${_scons_args} install_root=${DESTDIR}/usr install
-}
+distfiles="https://github.com/mixxxdj/mixxx/archive/${version}.tar.gz"
+checksum=8e3a5a545175982336bb07c81a3839897a007c43689b93641242db662f6beb9c

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: mixxx: update to 2.3.3.
  2022-08-02  0:34 [PR PATCH] mixxx: update to 2.3.3 abenson
  2022-08-02  2:12 ` [PR PATCH] [Updated] " abenson
@ 2022-08-02 13:26 ` prez
  2022-08-02 13:29 ` prez
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: prez @ 2022-08-02 13:26 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 699 bytes --]

New comment by prez on void-packages repository

https://github.com/void-linux/void-packages/pull/38416#issuecomment-1202557726

Comment:
I'm not sure why you chose to ignore https://github.com/void-linux/void-packages/pull/34808. 
Have you checked whether those dependencies are correct? In my PR, there's 
```
-libmp4v2-devel
-libshout-devel

+libflac-devel
+libmodplug-devel
+libogg-devel
+libsndfile-devel
+libvorbis-devel
+lilv-devel
+wavpack-devel
```
compared to yours.
It's been over half a year since I made that PR and I don't feel like re-checking all of it, but I must have had reasons. 

Additionally, is the patch needed? It seems to build fine on musl CI without it.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: mixxx: update to 2.3.3.
  2022-08-02  0:34 [PR PATCH] mixxx: update to 2.3.3 abenson
  2022-08-02  2:12 ` [PR PATCH] [Updated] " abenson
  2022-08-02 13:26 ` prez
@ 2022-08-02 13:29 ` prez
  2022-08-02 13:32 ` prez
  2022-08-02 18:04 ` [PR PATCH] [Closed]: " abenson
  4 siblings, 0 replies; 6+ messages in thread
From: prez @ 2022-08-02 13:29 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 686 bytes --]

New comment by prez on void-packages repository

https://github.com/void-linux/void-packages/pull/38416#issuecomment-1202557726

Comment:
I'm not sure why you chose to ignore https://github.com/void-linux/void-packages/pull/34808. 
Have you checked whether those dependencies are correct? In my PR, there's 
```
-libmp4v2-devel
-libshout-devel

+libflac-devel
+libmodplug-devel
+libogg-devel
+libsndfile-devel
+libvorbis-devel
+wavpack-devel
```
compared to yours.
It's been over half a year since I made that PR and I don't feel like re-checking all of it, but I must have had reasons. 

Additionally, is the patch needed? It seems to build fine on musl CI without it.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: mixxx: update to 2.3.3.
  2022-08-02  0:34 [PR PATCH] mixxx: update to 2.3.3 abenson
                   ` (2 preceding siblings ...)
  2022-08-02 13:29 ` prez
@ 2022-08-02 13:32 ` prez
  2022-08-02 18:04 ` [PR PATCH] [Closed]: " abenson
  4 siblings, 0 replies; 6+ messages in thread
From: prez @ 2022-08-02 13:32 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 927 bytes --]

New comment by prez on void-packages repository

https://github.com/void-linux/void-packages/pull/38416#issuecomment-1202557726

Comment:
I'm not sure why you chose to ignore https://github.com/void-linux/void-packages/pull/34808. 
Have you checked whether those dependencies are correct? In my PR, there's 
```
-libmp4v2-devel
-libshout-devel

+libflac-devel
+libmodplug-devel
+libogg-devel
+libsndfile-devel
+libvorbis-devel
+wavpack-devel
```
compared to yours.
It's been over half a year since I made that PR and I don't feel like re-checking all of it, but I must have had reasons. 

Additionally, is the patch needed? It seems to build fine on musl CI without it.

I can say with confidence that with my PR, all features of mixxx work as expected, whereas I'm not so sure with yours - e.g. there could be breakage in streaming functionality, since non-bundled libshout is known to be broken very often.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Closed]: mixxx: update to 2.3.3.
  2022-08-02  0:34 [PR PATCH] mixxx: update to 2.3.3 abenson
                   ` (3 preceding siblings ...)
  2022-08-02 13:32 ` prez
@ 2022-08-02 18:04 ` abenson
  4 siblings, 0 replies; 6+ messages in thread
From: abenson @ 2022-08-02 18:04 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 446 bytes --]

There's a closed pull request on the void-packages repository

mixxx: update to 2.3.3.
https://github.com/void-linux/void-packages/pull/38416

Description:

#### Testing the changes
- I tested the changes in this PR: **briefly**

#### Local build testing
- I built this PR locally for my native architecture, x86_64
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - x86_64-musl
  - aarch64 (cross)


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-08-02 18:04 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-02  0:34 [PR PATCH] mixxx: update to 2.3.3 abenson
2022-08-02  2:12 ` [PR PATCH] [Updated] " abenson
2022-08-02 13:26 ` prez
2022-08-02 13:29 ` prez
2022-08-02 13:32 ` prez
2022-08-02 18:04 ` [PR PATCH] [Closed]: " abenson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).