* [PR PATCH] webrtc-audio-processing: fix for i686-musl build
@ 2024-07-24 14:25 xeroxslayer
2024-07-24 14:29 ` [PR REVIEW] " classabbyamp
` (7 more replies)
0 siblings, 8 replies; 9+ messages in thread
From: xeroxslayer @ 2024-07-24 14:25 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 919 bytes --]
There is a new pull request by xeroxslayer against master on the void-packages repository
https://github.com/xeroxslayer/void-packages webrtc-audio-processing
https://github.com/void-linux/void-packages/pull/51424
webrtc-audio-processing: fix for i686-musl build
#### Testing the changes
- I tested the changes in this PR: **YES**
#### Local build testing
- I built this PR locally for my native architecture: x86_64-musl, i686-musl (with `-A i686-musl`)
- I built this PR locally for these architectures:
- x86_64 (`-A x86_64` on x86_64-musl)
- i686 (`-A i686` on x86_64-musl)
- aarch64 (`-A x86_64 -a aarch64` on x86_64-musl)
- aarch64-musl (`-A x86_64-musl -a aarch64-musl` on x86_64-musl)
- armv7l (`-A i686 -a armv7l` on x86_64-musl)
- armv7l-musl (`-A i686-musl -a armv7l-musl` on x86_64-musl)
A patch file from https://github.com/void-linux/void-packages/pull/51424.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-webrtc-audio-processing-51424.patch --]
[-- Type: text/x-diff, Size: 986 bytes --]
From 35d5d5260bc01998eb31957d4450e94788aa7927 Mon Sep 17 00:00:00 2001
From: xeroxslayer <xeroxslayer@proton.me>
Date: Wed, 24 Jul 2024 16:06:08 +0200
Subject: [PATCH] Fix i686-musl build
---
srcpkgs/webrtc-audio-processing/template | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/srcpkgs/webrtc-audio-processing/template b/srcpkgs/webrtc-audio-processing/template
index e5277d17e0b9e3..5ee8c56f0ac173 100644
--- a/srcpkgs/webrtc-audio-processing/template
+++ b/srcpkgs/webrtc-audio-processing/template
@@ -13,7 +13,7 @@ distfiles="${FREEDESKTOP_SITE}/pulseaudio/webrtc-audio-processing/webrtc-audio-p
checksum=95552fc17faa0202133707bbb3727e8c2cf64d4266fe31bfdb2298d769c1db75
# Upstream issue: https://gitlab.freedesktop.org/pulseaudio/webrtc-audio-processing/-/issues/5
-if [ "$XBPS_MACHINE" = "i686" ]; then
+if [ "$XBPS_MACHINE" = "i686" ] || [ "$XBPS_MACHINE" = "i686-musl" ]; then
CXXFLAGS="-DPFFFT_SIMD_DISABLE=1"
CFLAGS="-DPFFFT_SIMD_DISABLE=1"
fi
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR REVIEW] webrtc-audio-processing: fix for i686-musl build
2024-07-24 14:25 [PR PATCH] webrtc-audio-processing: fix for i686-musl build xeroxslayer
@ 2024-07-24 14:29 ` classabbyamp
2024-07-24 14:34 ` [PR PATCH] [Updated] " xeroxslayer
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: classabbyamp @ 2024-07-24 14:29 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 220 bytes --]
New review comment by classabbyamp on void-packages repository
https://github.com/void-linux/void-packages/pull/51424#discussion_r1689927798
Comment:
```suggestion
if [[ "$XBPS_TARGET_MACHINE" = "i686"* ]]; then
```
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR PATCH] [Updated] webrtc-audio-processing: fix for i686-musl build
2024-07-24 14:25 [PR PATCH] webrtc-audio-processing: fix for i686-musl build xeroxslayer
2024-07-24 14:29 ` [PR REVIEW] " classabbyamp
@ 2024-07-24 14:34 ` xeroxslayer
2024-07-24 14:35 ` [PR REVIEW] " abenson
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: xeroxslayer @ 2024-07-24 14:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 924 bytes --]
There is an updated pull request by xeroxslayer against master on the void-packages repository
https://github.com/xeroxslayer/void-packages webrtc-audio-processing
https://github.com/void-linux/void-packages/pull/51424
webrtc-audio-processing: fix for i686-musl build
#### Testing the changes
- I tested the changes in this PR: **YES**
#### Local build testing
- I built this PR locally for my native architecture: x86_64-musl, i686-musl (with `-A i686-musl`)
- I built this PR locally for these architectures:
- x86_64 (`-A x86_64` on x86_64-musl)
- i686 (`-A i686` on x86_64-musl)
- aarch64 (`-A x86_64 -a aarch64` on x86_64-musl)
- aarch64-musl (`-A x86_64-musl -a aarch64-musl` on x86_64-musl)
- armv7l (`-A i686 -a armv7l` on x86_64-musl)
- armv7l-musl (`-A i686-musl -a armv7l-musl` on x86_64-musl)
A patch file from https://github.com/void-linux/void-packages/pull/51424.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-webrtc-audio-processing-51424.patch --]
[-- Type: text/x-diff, Size: 2111 bytes --]
From 35d5d5260bc01998eb31957d4450e94788aa7927 Mon Sep 17 00:00:00 2001
From: xeroxslayer <xeroxslayer@proton.me>
Date: Wed, 24 Jul 2024 16:06:08 +0200
Subject: [PATCH 1/2] Fix i686-musl build
---
srcpkgs/webrtc-audio-processing/template | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/srcpkgs/webrtc-audio-processing/template b/srcpkgs/webrtc-audio-processing/template
index e5277d17e0b9e3..5ee8c56f0ac173 100644
--- a/srcpkgs/webrtc-audio-processing/template
+++ b/srcpkgs/webrtc-audio-processing/template
@@ -13,7 +13,7 @@ distfiles="${FREEDESKTOP_SITE}/pulseaudio/webrtc-audio-processing/webrtc-audio-p
checksum=95552fc17faa0202133707bbb3727e8c2cf64d4266fe31bfdb2298d769c1db75
# Upstream issue: https://gitlab.freedesktop.org/pulseaudio/webrtc-audio-processing/-/issues/5
-if [ "$XBPS_MACHINE" = "i686" ]; then
+if [ "$XBPS_MACHINE" = "i686" ] || [ "$XBPS_MACHINE" = "i686-musl" ]; then
CXXFLAGS="-DPFFFT_SIMD_DISABLE=1"
CFLAGS="-DPFFFT_SIMD_DISABLE=1"
fi
From cab7f975c0f7bd71525f9560f2208180871bb1ad Mon Sep 17 00:00:00 2001
From: xeroxslayer <xeroxslayer@proton.me>
Date: Wed, 24 Jul 2024 16:34:44 +0200
Subject: [PATCH 2/2] Update srcpkgs/webrtc-audio-processing/template
A better fix.
Co-authored-by: classabbyamp <5366828+classabbyamp@users.noreply.github.com>
---
srcpkgs/webrtc-audio-processing/template | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/srcpkgs/webrtc-audio-processing/template b/srcpkgs/webrtc-audio-processing/template
index 5ee8c56f0ac173..e8d6f805c35840 100644
--- a/srcpkgs/webrtc-audio-processing/template
+++ b/srcpkgs/webrtc-audio-processing/template
@@ -13,7 +13,7 @@ distfiles="${FREEDESKTOP_SITE}/pulseaudio/webrtc-audio-processing/webrtc-audio-p
checksum=95552fc17faa0202133707bbb3727e8c2cf64d4266fe31bfdb2298d769c1db75
# Upstream issue: https://gitlab.freedesktop.org/pulseaudio/webrtc-audio-processing/-/issues/5
-if [ "$XBPS_MACHINE" = "i686" ] || [ "$XBPS_MACHINE" = "i686-musl" ]; then
+if [[ "$XBPS_TARGET_MACHINE" = "i686"* ]]; then
CXXFLAGS="-DPFFFT_SIMD_DISABLE=1"
CFLAGS="-DPFFFT_SIMD_DISABLE=1"
fi
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR REVIEW] webrtc-audio-processing: fix for i686-musl build
2024-07-24 14:25 [PR PATCH] webrtc-audio-processing: fix for i686-musl build xeroxslayer
2024-07-24 14:29 ` [PR REVIEW] " classabbyamp
2024-07-24 14:34 ` [PR PATCH] [Updated] " xeroxslayer
@ 2024-07-24 14:35 ` abenson
2024-07-24 14:48 ` abenson
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: abenson @ 2024-07-24 14:35 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 391 bytes --]
New review comment by abenson on void-packages repository
https://github.com/void-linux/void-packages/pull/51424#discussion_r1689938024
Comment:
In addition to classabbyamp's suggestion, please fix the commit message to follow the convention. The message used at the PR's title will suffice, though you could amend it to just say "for i686" since it seems to apply to both i686/i686-musl.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR REVIEW] webrtc-audio-processing: fix for i686-musl build
2024-07-24 14:25 [PR PATCH] webrtc-audio-processing: fix for i686-musl build xeroxslayer
` (2 preceding siblings ...)
2024-07-24 14:35 ` [PR REVIEW] " abenson
@ 2024-07-24 14:48 ` abenson
2024-07-24 20:40 ` xeroxslayer
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: abenson @ 2024-07-24 14:48 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 305 bytes --]
New review comment by abenson on void-packages repository
https://github.com/void-linux/void-packages/pull/51424#discussion_r1689938024
Comment:
In addition to classabbyamp's suggestion, please squash and fix the commit message to follow the convention. The message used at the PR's title will suffice.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR REVIEW] webrtc-audio-processing: fix for i686-musl build
2024-07-24 14:25 [PR PATCH] webrtc-audio-processing: fix for i686-musl build xeroxslayer
` (3 preceding siblings ...)
2024-07-24 14:48 ` abenson
@ 2024-07-24 20:40 ` xeroxslayer
2024-07-25 10:47 ` [PR PATCH] [Updated] " xeroxslayer
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: xeroxslayer @ 2024-07-24 20:40 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 160 bytes --]
New review comment by xeroxslayer on void-packages repository
https://github.com/void-linux/void-packages/pull/51424#discussion_r1690397383
Comment:
Will do.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR PATCH] [Updated] webrtc-audio-processing: fix for i686-musl build
2024-07-24 14:25 [PR PATCH] webrtc-audio-processing: fix for i686-musl build xeroxslayer
` (4 preceding siblings ...)
2024-07-24 20:40 ` xeroxslayer
@ 2024-07-25 10:47 ` xeroxslayer
2024-07-25 10:53 ` [PR PATCH] [Closed]: " xeroxslayer
2024-07-25 10:53 ` [PR PATCH] [Updated] " xeroxslayer
7 siblings, 0 replies; 9+ messages in thread
From: xeroxslayer @ 2024-07-25 10:47 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 924 bytes --]
There is an updated pull request by xeroxslayer against master on the void-packages repository
https://github.com/xeroxslayer/void-packages webrtc-audio-processing
https://github.com/void-linux/void-packages/pull/51424
webrtc-audio-processing: fix for i686-musl build
#### Testing the changes
- I tested the changes in this PR: **YES**
#### Local build testing
- I built this PR locally for my native architecture: x86_64-musl, i686-musl (with `-A i686-musl`)
- I built this PR locally for these architectures:
- x86_64 (`-A x86_64` on x86_64-musl)
- i686 (`-A i686` on x86_64-musl)
- aarch64 (`-A x86_64 -a aarch64` on x86_64-musl)
- aarch64-musl (`-A x86_64-musl -a aarch64-musl` on x86_64-musl)
- armv7l (`-A i686 -a armv7l` on x86_64-musl)
- armv7l-musl (`-A i686-musl -a armv7l-musl` on x86_64-musl)
A patch file from https://github.com/void-linux/void-packages/pull/51424.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-webrtc-audio-processing-51424.patch --]
[-- Type: text/x-diff, Size: 2111 bytes --]
From 35d5d5260bc01998eb31957d4450e94788aa7927 Mon Sep 17 00:00:00 2001
From: xeroxslayer <xeroxslayer@proton.me>
Date: Wed, 24 Jul 2024 16:06:08 +0200
Subject: [PATCH 1/2] Fix i686-musl build
---
srcpkgs/webrtc-audio-processing/template | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/srcpkgs/webrtc-audio-processing/template b/srcpkgs/webrtc-audio-processing/template
index e5277d17e0b9e3..5ee8c56f0ac173 100644
--- a/srcpkgs/webrtc-audio-processing/template
+++ b/srcpkgs/webrtc-audio-processing/template
@@ -13,7 +13,7 @@ distfiles="${FREEDESKTOP_SITE}/pulseaudio/webrtc-audio-processing/webrtc-audio-p
checksum=95552fc17faa0202133707bbb3727e8c2cf64d4266fe31bfdb2298d769c1db75
# Upstream issue: https://gitlab.freedesktop.org/pulseaudio/webrtc-audio-processing/-/issues/5
-if [ "$XBPS_MACHINE" = "i686" ]; then
+if [ "$XBPS_MACHINE" = "i686" ] || [ "$XBPS_MACHINE" = "i686-musl" ]; then
CXXFLAGS="-DPFFFT_SIMD_DISABLE=1"
CFLAGS="-DPFFFT_SIMD_DISABLE=1"
fi
From cab7f975c0f7bd71525f9560f2208180871bb1ad Mon Sep 17 00:00:00 2001
From: xeroxslayer <xeroxslayer@proton.me>
Date: Wed, 24 Jul 2024 16:34:44 +0200
Subject: [PATCH 2/2] Update srcpkgs/webrtc-audio-processing/template
A better fix.
Co-authored-by: classabbyamp <5366828+classabbyamp@users.noreply.github.com>
---
srcpkgs/webrtc-audio-processing/template | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/srcpkgs/webrtc-audio-processing/template b/srcpkgs/webrtc-audio-processing/template
index 5ee8c56f0ac173..e8d6f805c35840 100644
--- a/srcpkgs/webrtc-audio-processing/template
+++ b/srcpkgs/webrtc-audio-processing/template
@@ -13,7 +13,7 @@ distfiles="${FREEDESKTOP_SITE}/pulseaudio/webrtc-audio-processing/webrtc-audio-p
checksum=95552fc17faa0202133707bbb3727e8c2cf64d4266fe31bfdb2298d769c1db75
# Upstream issue: https://gitlab.freedesktop.org/pulseaudio/webrtc-audio-processing/-/issues/5
-if [ "$XBPS_MACHINE" = "i686" ] || [ "$XBPS_MACHINE" = "i686-musl" ]; then
+if [[ "$XBPS_TARGET_MACHINE" = "i686"* ]]; then
CXXFLAGS="-DPFFFT_SIMD_DISABLE=1"
CFLAGS="-DPFFFT_SIMD_DISABLE=1"
fi
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR PATCH] [Closed]: webrtc-audio-processing: fix for i686-musl build
2024-07-24 14:25 [PR PATCH] webrtc-audio-processing: fix for i686-musl build xeroxslayer
` (5 preceding siblings ...)
2024-07-25 10:47 ` [PR PATCH] [Updated] " xeroxslayer
@ 2024-07-25 10:53 ` xeroxslayer
2024-07-25 10:53 ` [PR PATCH] [Updated] " xeroxslayer
7 siblings, 0 replies; 9+ messages in thread
From: xeroxslayer @ 2024-07-25 10:53 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 744 bytes --]
There's a closed pull request on the void-packages repository
webrtc-audio-processing: fix for i686-musl build
https://github.com/void-linux/void-packages/pull/51424
Description:
#### Testing the changes
- I tested the changes in this PR: **YES**
#### Local build testing
- I built this PR locally for my native architecture: x86_64-musl, i686-musl (with `-A i686-musl`)
- I built this PR locally for these architectures:
- x86_64 (`-A x86_64` on x86_64-musl)
- i686 (`-A i686` on x86_64-musl)
- aarch64 (`-A x86_64 -a aarch64` on x86_64-musl)
- aarch64-musl (`-A x86_64-musl -a aarch64-musl` on x86_64-musl)
- armv7l (`-A i686 -a armv7l` on x86_64-musl)
- armv7l-musl (`-A i686-musl -a armv7l-musl` on x86_64-musl)
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR PATCH] [Updated] webrtc-audio-processing: fix for i686-musl build
2024-07-24 14:25 [PR PATCH] webrtc-audio-processing: fix for i686-musl build xeroxslayer
` (6 preceding siblings ...)
2024-07-25 10:53 ` [PR PATCH] [Closed]: " xeroxslayer
@ 2024-07-25 10:53 ` xeroxslayer
7 siblings, 0 replies; 9+ messages in thread
From: xeroxslayer @ 2024-07-25 10:53 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 924 bytes --]
There is an updated pull request by xeroxslayer against master on the void-packages repository
https://github.com/xeroxslayer/void-packages webrtc-audio-processing
https://github.com/void-linux/void-packages/pull/51424
webrtc-audio-processing: fix for i686-musl build
#### Testing the changes
- I tested the changes in this PR: **YES**
#### Local build testing
- I built this PR locally for my native architecture: x86_64-musl, i686-musl (with `-A i686-musl`)
- I built this PR locally for these architectures:
- x86_64 (`-A x86_64` on x86_64-musl)
- i686 (`-A i686` on x86_64-musl)
- aarch64 (`-A x86_64 -a aarch64` on x86_64-musl)
- aarch64-musl (`-A x86_64-musl -a aarch64-musl` on x86_64-musl)
- armv7l (`-A i686 -a armv7l` on x86_64-musl)
- armv7l-musl (`-A i686-musl -a armv7l-musl` on x86_64-musl)
A patch file from https://github.com/void-linux/void-packages/pull/51424.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-webrtc-audio-processing-51424.patch --]
[-- Type: text/x-diff, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2024-07-25 10:53 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-07-24 14:25 [PR PATCH] webrtc-audio-processing: fix for i686-musl build xeroxslayer
2024-07-24 14:29 ` [PR REVIEW] " classabbyamp
2024-07-24 14:34 ` [PR PATCH] [Updated] " xeroxslayer
2024-07-24 14:35 ` [PR REVIEW] " abenson
2024-07-24 14:48 ` abenson
2024-07-24 20:40 ` xeroxslayer
2024-07-25 10:47 ` [PR PATCH] [Updated] " xeroxslayer
2024-07-25 10:53 ` [PR PATCH] [Closed]: " xeroxslayer
2024-07-25 10:53 ` [PR PATCH] [Updated] " xeroxslayer
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).