[PR PATCH] mpg123{,-extras}: update to 1.31.3.

[PR PATCH] mpg123{,-extras}: update to 1.31.3.

[mhmdanas]

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

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

https://github.com/mhmdanas/void-packages mpg123-1.31.3
https://github.com/void-linux/void-packages/pull/44463

mpg123{,-extras}: update to 1.31.3.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **briefly** (not mpg123-extras and its subpackages though, because I couldn't find any package that depends on them)

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mpg123-1.31.3-44463.patch --]
[-- Type: text/x-diff, Size: 3765 bytes --]

From 82de0036914883afae01d151d98cadaa87eb396a Mon Sep 17 00:00:00 2001
From: triallax <triallax@tutanota.com>
Date: Fri, 16 Jun 2023 10:12:43 +0100
Subject: [PATCH 1/2] mpg123: update to 1.31.3.

---
 srcpkgs/mpg123/template | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/mpg123/template b/srcpkgs/mpg123/template
index 03bf482ef453..12960dcb131a 100644
--- a/srcpkgs/mpg123/template
+++ b/srcpkgs/mpg123/template
@@ -1,22 +1,20 @@
 # Template file for 'mpg123'
 # Sync with mpg123-extras
 pkgname=mpg123
-version=1.31.2
+version=1.31.3
 revision=1
 build_style=gnu-configure
 # --with-cpu not necessary, upstream detects features correctly
-configure_args="--with-optimization=0 --with-default-audio=alsa"
+configure_args="--with-default-audio=alsa"
 hostmakedepends="pkg-config"
 makedepends="alsa-lib-devel"
 short_desc="Fast console MPEG audio decoder/player"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="LGPL-2.1-only"
 homepage="https://www.mpg123.org/"
-changelog="https://www.mpg123.org/cgi-bin/scm/mpg123/trunk/NEWS"
-distfiles="${SOURCEFORGE_SITE}/mpg123/mpg123-${version}.tar.bz2
- https://www.mpg123.org/download/mpg123-${version}.tar.bz2"
-checksum="b17f22905e31f43b6b401dfdf6a71ed11bb7d056f68db449d70b9f9ae839c7de
- b17f22905e31f43b6b401dfdf6a71ed11bb7d056f68db449d70b9f9ae839c7de"
+changelog="https://www.mpg123.de/trunk/NEWS"
+distfiles="${SOURCEFORGE_SITE}/mpg123/mpg123-${version}.tar.bz2"
+checksum=1ca77d3a69a5ff845b7a0536f783fee554e1041139a6b978f6afe14f5814ad1a
 
 if [ "$XBPS_TARGET_LIBC" = musl ]; then
 	# No LFS required with musl

From 5866b4e5ebda96d515792251969312d66766aaf1 Mon Sep 17 00:00:00 2001
From: triallax <triallax@tutanota.com>
Date: Fri, 16 Jun 2023 10:58:44 +0100
Subject: [PATCH 2/2] mpg123-extras: update to 1.31.3.

---
 srcpkgs/mpg123-extras/template | 12 +++++-------
 srcpkgs/mpg123-extras/update   |  1 +
 2 files changed, 6 insertions(+), 7 deletions(-)
 create mode 100644 srcpkgs/mpg123-extras/update

diff --git a/srcpkgs/mpg123-extras/template b/srcpkgs/mpg123-extras/template
index 0c7795b73873..1b1e2e13cdb7 100644
--- a/srcpkgs/mpg123-extras/template
+++ b/srcpkgs/mpg123-extras/template
@@ -1,11 +1,11 @@
 # Template file for 'mpg123-extras'
 # Sync with mpg123
 pkgname=mpg123-extras
-version=1.31.2
+version=1.31.3
 revision=1
 build_style=gnu-configure
 # --with-cpu not necessary, upstream detects features correctly
-configure_args="--with-optimization=0 --with-default-audio=alsa"
+configure_args="--with-default-audio=alsa"
 hostmakedepends="pkg-config"
 makedepends="alsa-lib-devel SDL2-devel jack-devel pulseaudio-devel sndio-devel"
 depends="mpg123-sdl mpg123-jack mpg123-pulseaudio mpg123-sndio"
@@ -13,11 +13,9 @@ short_desc="Fast console MPEG audio decoder/player - extra plugins"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="LGPL-2.1-only"
 homepage="https://www.mpg123.org/"
-changelog="https://www.mpg123.org/cgi-bin/scm/mpg123/trunk/NEWS"
-distfiles="${SOURCEFORGE_SITE}/mpg123/mpg123-${version}.tar.bz2
- https://www.mpg123.org/download/mpg123-${version}.tar.bz2"
-checksum="b17f22905e31f43b6b401dfdf6a71ed11bb7d056f68db449d70b9f9ae839c7de
- b17f22905e31f43b6b401dfdf6a71ed11bb7d056f68db449d70b9f9ae839c7de"
+changelog="https://www.mpg123.de/trunk/NEWS"
+distfiles="https://www.mpg123.org/download/mpg123-${version}.tar.bz2"
+checksum=1ca77d3a69a5ff845b7a0536f783fee554e1041139a6b978f6afe14f5814ad1a
 
 # Restrict extras to the same matching version, ignore revision
 _depends="libmpg123>=${version}_1<=${version}_9999"
diff --git a/srcpkgs/mpg123-extras/update b/srcpkgs/mpg123-extras/update
new file mode 100644
index 000000000000..3bd50d571a0a
--- /dev/null
+++ b/srcpkgs/mpg123-extras/update
@@ -0,0 +1 @@
+pkgname=mpg123

Re: [PR REVIEW] mpg123{,-extras}: update to 1.31.3.

[Duncaen]

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

New review comment by Duncaen on void-packages repository

https://github.com/void-linux/void-packages/pull/44463#discussion_r1238349802

Comment:
Pretty sure we set `--with-optimization=0` so that it respects the default of `xbps-src`.

Re: [PR PATCH] [Updated] mpg123{,-extras}: update to 1.31.3.

[mhmdanas]

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

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

https://github.com/mhmdanas/void-packages mpg123-1.31.3
https://github.com/void-linux/void-packages/pull/44463

mpg123{,-extras}: update to 1.31.3.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **briefly** (not mpg123-extras and its subpackages though, because I couldn't find any package that depends on them)

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mpg123-1.31.3-44463.patch --]
[-- Type: text/x-diff, Size: 3790 bytes --]

From 99b9727e09dfced45832d2fcc690d4303584163a Mon Sep 17 00:00:00 2001
From: triallax <triallax@tutanota.com>
Date: Fri, 16 Jun 2023 10:12:43 +0100
Subject: [PATCH 1/2] mpg123: update to 1.31.3.

---
 srcpkgs/mpg123/template | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/mpg123/template b/srcpkgs/mpg123/template
index 03bf482ef453..90823cec2e18 100644
--- a/srcpkgs/mpg123/template
+++ b/srcpkgs/mpg123/template
@@ -1,10 +1,11 @@
 # Template file for 'mpg123'
 # Sync with mpg123-extras
 pkgname=mpg123
-version=1.31.2
+version=1.31.3
 revision=1
 build_style=gnu-configure
 # --with-cpu not necessary, upstream detects features correctly
+# --with-optimization=0 makes the configure script respect xbps-src's defaults.
 configure_args="--with-optimization=0 --with-default-audio=alsa"
 hostmakedepends="pkg-config"
 makedepends="alsa-lib-devel"
@@ -12,11 +13,9 @@ short_desc="Fast console MPEG audio decoder/player"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="LGPL-2.1-only"
 homepage="https://www.mpg123.org/"
-changelog="https://www.mpg123.org/cgi-bin/scm/mpg123/trunk/NEWS"
-distfiles="${SOURCEFORGE_SITE}/mpg123/mpg123-${version}.tar.bz2
- https://www.mpg123.org/download/mpg123-${version}.tar.bz2"
-checksum="b17f22905e31f43b6b401dfdf6a71ed11bb7d056f68db449d70b9f9ae839c7de
- b17f22905e31f43b6b401dfdf6a71ed11bb7d056f68db449d70b9f9ae839c7de"
+changelog="https://www.mpg123.de/trunk/NEWS"
+distfiles="${SOURCEFORGE_SITE}/mpg123/mpg123-${version}.tar.bz2"
+checksum=1ca77d3a69a5ff845b7a0536f783fee554e1041139a6b978f6afe14f5814ad1a
 
 if [ "$XBPS_TARGET_LIBC" = musl ]; then
 	# No LFS required with musl

From b910447e4b8a436ec949d9c240e26a1344d13635 Mon Sep 17 00:00:00 2001
From: triallax <triallax@tutanota.com>
Date: Fri, 16 Jun 2023 10:58:44 +0100
Subject: [PATCH 2/2] mpg123-extras: update to 1.31.3.

---
 srcpkgs/mpg123-extras/template | 11 +++++------
 srcpkgs/mpg123-extras/update   |  1 +
 2 files changed, 6 insertions(+), 6 deletions(-)
 create mode 100644 srcpkgs/mpg123-extras/update

diff --git a/srcpkgs/mpg123-extras/template b/srcpkgs/mpg123-extras/template
index 0c7795b73873..3229619548a0 100644
--- a/srcpkgs/mpg123-extras/template
+++ b/srcpkgs/mpg123-extras/template
@@ -1,10 +1,11 @@
 # Template file for 'mpg123-extras'
 # Sync with mpg123
 pkgname=mpg123-extras
-version=1.31.2
+version=1.31.3
 revision=1
 build_style=gnu-configure
 # --with-cpu not necessary, upstream detects features correctly
+# --with-optimization=0 makes the configure script respect xbps-src's defaults.
 configure_args="--with-optimization=0 --with-default-audio=alsa"
 hostmakedepends="pkg-config"
 makedepends="alsa-lib-devel SDL2-devel jack-devel pulseaudio-devel sndio-devel"
@@ -13,11 +14,9 @@ short_desc="Fast console MPEG audio decoder/player - extra plugins"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="LGPL-2.1-only"
 homepage="https://www.mpg123.org/"
-changelog="https://www.mpg123.org/cgi-bin/scm/mpg123/trunk/NEWS"
-distfiles="${SOURCEFORGE_SITE}/mpg123/mpg123-${version}.tar.bz2
- https://www.mpg123.org/download/mpg123-${version}.tar.bz2"
-checksum="b17f22905e31f43b6b401dfdf6a71ed11bb7d056f68db449d70b9f9ae839c7de
- b17f22905e31f43b6b401dfdf6a71ed11bb7d056f68db449d70b9f9ae839c7de"
+changelog="https://www.mpg123.de/trunk/NEWS"
+distfiles="https://www.mpg123.org/download/mpg123-${version}.tar.bz2"
+checksum=1ca77d3a69a5ff845b7a0536f783fee554e1041139a6b978f6afe14f5814ad1a
 
 # Restrict extras to the same matching version, ignore revision
 _depends="libmpg123>=${version}_1<=${version}_9999"
diff --git a/srcpkgs/mpg123-extras/update b/srcpkgs/mpg123-extras/update
new file mode 100644
index 000000000000..3bd50d571a0a
--- /dev/null
+++ b/srcpkgs/mpg123-extras/update
@@ -0,0 +1 @@
+pkgname=mpg123

Re: [PR REVIEW] mpg123{,-extras}: update to 1.31.3.

[mhmdanas]

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

New review comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/pull/44463#discussion_r1252930493

Comment:
Thanks for the heads-up, I put it back with an explanatory comment.

Re: [PR PATCH] [Merged]: mpg123{,-extras}: update to 1.31.3.

[leahneukirchen]

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

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

mpg123{,-extras}: update to 1.31.3.
https://github.com/void-linux/void-packages/pull/44463

Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **briefly** (not mpg123-extras and its subpackages though, because I couldn't find any package that depends on them)

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->

Re: [PR REVIEW] mpg123{,-extras}: update to 1.31.3.

[leahneukirchen]

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

New review comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/44463#discussion_r1285109844

Comment:
ftr, this link doesnt work

Re: [PR REVIEW] mpg123{,-extras}: update to 1.31.3.

[mhmdanas]

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

New review comment by mhmdanas on void-packages repository

https://github.com/void-linux/void-packages/pull/44463#discussion_r1285112592

Comment:
Yeah, that's why I changed it to something that does.

Or do you mean something different?

Re: [PR REVIEW] mpg123{,-extras}: update to 1.31.3.

[leahneukirchen]

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

New review comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/44463#discussion_r1285112620

Comment:
oh it was deleted >_<