Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] pipewire: remove libpulseaudio-pipewire from repos.
@ 2021-07-25 20:33 JamiKettunen
  2021-07-25 20:40 ` [PR PATCH] [Updated] " JamiKettunen
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: JamiKettunen @ 2021-07-25 20:33 UTC (permalink / raw)
  To: ml

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

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

https://github.com/JamiKettunen/void-packages no-libpulseaudio-pipewire
https://github.com/void-linux/void-packages/pull/32200

pipewire: remove libpulseaudio-pipewire from repos.
The packages for this are [still around on repos](https://voidlinux.org/packages/?arch=x86_64&q=libpulseaudio-pipewire), I think it's about time for those to go.
This subpackage [was deprecated with the upgrade to PipeWire 0.3.18](https://github.com/void-linux/void-packages/pull/27279).

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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/32200.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-no-libpulseaudio-pipewire-32200.patch --]
[-- Type: text/x-diff, Size: 1822 bytes --]

From 31b698e9ef7bb87a7d33e8091765f9d12a8151ec Mon Sep 17 00:00:00 2001
From: Jami Kettunen <jami.kettunen@protonmail.com>
Date: Sun, 25 Jul 2021 23:10:57 +0300
Subject: [PATCH] pipewire: remove libpulseaudio-pipewire from repos.

The packages for this are still around on repos, I think it's about time
for those to go.
---
 srcpkgs/pipewire/template         | 4 +---
 srcpkgs/removed-packages/template | 3 ++-
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/pipewire/template b/srcpkgs/pipewire/template
index b61c6a74e63e..ffa715e26373 100644
--- a/srcpkgs/pipewire/template
+++ b/srcpkgs/pipewire/template
@@ -1,7 +1,7 @@
 # Template file for 'pipewire'
 pkgname=pipewire
 version=0.3.32
-revision=1
+revision=2
 build_style=meson
 configure_args="-Dman=enabled -Dgstreamer=enabled -Ddocs=enabled -Dsystemd=disabled
  -Dbluez5=enabled -Dffmpeg=enabled -Dpipewire-alsa=enabled -Dpipewire-jack=enabled
@@ -24,8 +24,6 @@ system_accounts="_pipewire"
 
 build_options="sdl2"
 
-replaces="libpulseaudio-pipewire>=0"
-
 if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
 	makedepends+=" libatomic-devel"
 	LDFLAGS+=" -latomic"
diff --git a/srcpkgs/removed-packages/template b/srcpkgs/removed-packages/template
index 6720b1724742..4f2ffaaf1997 100644
--- a/srcpkgs/removed-packages/template
+++ b/srcpkgs/removed-packages/template
@@ -1,7 +1,7 @@
 # Template file for 'removed-packages'
 pkgname=removed-packages
 version=0.1
-revision=46
+revision=47
 build_style=meta
 short_desc="Uninstalls packages removed from repository"
 maintainer="Piotr Wójcik <chocimier@tlen.pl>"
@@ -194,6 +194,7 @@ replaces="
  libfcitx-qt-devel<=4.2.9.8_1
  libgksu<=2.0.12_5
  libglib-static<=2.58.3_5
+ libpulseaudio-pipewire<=0.3.17
  libpyside-python3<=5.15.0_2
  libqmatrixclient<=0.5.3.2_1
  libqmatrixclient-devel<=0.5.3.2_1

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

* Re: [PR PATCH] [Updated] pipewire: remove libpulseaudio-pipewire from repos.
  2021-07-25 20:33 [PR PATCH] pipewire: remove libpulseaudio-pipewire from repos JamiKettunen
@ 2021-07-25 20:40 ` JamiKettunen
  2021-07-25 20:41 ` JamiKettunen
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: JamiKettunen @ 2021-07-25 20:40 UTC (permalink / raw)
  To: ml

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

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

https://github.com/JamiKettunen/void-packages no-libpulseaudio-pipewire
https://github.com/void-linux/void-packages/pull/32200

pipewire: remove libpulseaudio-pipewire from repos.
The packages for this are [still around on repos](https://voidlinux.org/packages/?arch=x86_64&q=libpulseaudio-pipewire), I think it's about time for those to go.
This subpackage [was deprecated with the upgrade to PipeWire 0.3.18](https://github.com/void-linux/void-packages/pull/27279).

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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/32200.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-no-libpulseaudio-pipewire-32200.patch --]
[-- Type: text/x-diff, Size: 1824 bytes --]

From 4616e63bc2da9a1cd84eef4355855d1f755baa44 Mon Sep 17 00:00:00 2001
From: Jami Kettunen <jami.kettunen@protonmail.com>
Date: Sun, 25 Jul 2021 23:10:57 +0300
Subject: [PATCH] pipewire: remove libpulseaudio-pipewire from repos.

The packages for this are still around on repos, I think it's about time
for those to go.
---
 srcpkgs/pipewire/template         | 4 +---
 srcpkgs/removed-packages/template | 3 ++-
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/pipewire/template b/srcpkgs/pipewire/template
index b61c6a74e63e..ffa715e26373 100644
--- a/srcpkgs/pipewire/template
+++ b/srcpkgs/pipewire/template
@@ -1,7 +1,7 @@
 # Template file for 'pipewire'
 pkgname=pipewire
 version=0.3.32
-revision=1
+revision=2
 build_style=meson
 configure_args="-Dman=enabled -Dgstreamer=enabled -Ddocs=enabled -Dsystemd=disabled
  -Dbluez5=enabled -Dffmpeg=enabled -Dpipewire-alsa=enabled -Dpipewire-jack=enabled
@@ -24,8 +24,6 @@ system_accounts="_pipewire"
 
 build_options="sdl2"
 
-replaces="libpulseaudio-pipewire>=0"
-
 if [ "$XBPS_TARGET_NO_ATOMIC8" ]; then
 	makedepends+=" libatomic-devel"
 	LDFLAGS+=" -latomic"
diff --git a/srcpkgs/removed-packages/template b/srcpkgs/removed-packages/template
index 6720b1724742..471c0a245028 100644
--- a/srcpkgs/removed-packages/template
+++ b/srcpkgs/removed-packages/template
@@ -1,7 +1,7 @@
 # Template file for 'removed-packages'
 pkgname=removed-packages
 version=0.1
-revision=46
+revision=47
 build_style=meta
 short_desc="Uninstalls packages removed from repository"
 maintainer="Piotr Wójcik <chocimier@tlen.pl>"
@@ -194,6 +194,7 @@ replaces="
  libfcitx-qt-devel<=4.2.9.8_1
  libgksu<=2.0.12_5
  libglib-static<=2.58.3_5
+ libpulseaudio-pipewire<=0.3.17_1
  libpyside-python3<=5.15.0_2
  libqmatrixclient<=0.5.3.2_1
  libqmatrixclient-devel<=0.5.3.2_1

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

* Re: pipewire: remove libpulseaudio-pipewire from repos.
  2021-07-25 20:33 [PR PATCH] pipewire: remove libpulseaudio-pipewire from repos JamiKettunen
  2021-07-25 20:40 ` [PR PATCH] [Updated] " JamiKettunen
@ 2021-07-25 20:41 ` JamiKettunen
  2021-07-25 20:50 ` paper42
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: JamiKettunen @ 2021-07-25 20:41 UTC (permalink / raw)
  To: ml

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

New comment by JamiKettunen on void-packages repository

https://github.com/void-linux/void-packages/pull/32200#issuecomment-886255056

Comment:
Ok I feel really dumb right now: I forgot to add the `_1` revision part to the `removed-packages` addition; fixed and force pushed.

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

* Re: pipewire: remove libpulseaudio-pipewire from repos.
  2021-07-25 20:33 [PR PATCH] pipewire: remove libpulseaudio-pipewire from repos JamiKettunen
  2021-07-25 20:40 ` [PR PATCH] [Updated] " JamiKettunen
  2021-07-25 20:41 ` JamiKettunen
@ 2021-07-25 20:50 ` paper42
  2021-07-25 21:01 ` ericonr
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: paper42 @ 2021-07-25 20:50 UTC (permalink / raw)
  To: ml

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

New comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/32200#issuecomment-886255982

Comment:
I think should be a comment on https://github.com/void-linux/void-packages/issues/23575

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

* Re: pipewire: remove libpulseaudio-pipewire from repos.
  2021-07-25 20:33 [PR PATCH] pipewire: remove libpulseaudio-pipewire from repos JamiKettunen
                   ` (2 preceding siblings ...)
  2021-07-25 20:50 ` paper42
@ 2021-07-25 21:01 ` ericonr
  2021-07-25 21:01 ` [PR PATCH] [Closed]: " ericonr
  2021-07-25 21:09 ` JamiKettunen
  5 siblings, 0 replies; 7+ messages in thread
From: ericonr @ 2021-07-25 21:01 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/32200#issuecomment-886257147

Comment:
What @paper42 said; the issue is the package can still be pulled from repos, not that it's locally installed for someone.

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

* Re: [PR PATCH] [Closed]: pipewire: remove libpulseaudio-pipewire from repos.
  2021-07-25 20:33 [PR PATCH] pipewire: remove libpulseaudio-pipewire from repos JamiKettunen
                   ` (3 preceding siblings ...)
  2021-07-25 21:01 ` ericonr
@ 2021-07-25 21:01 ` ericonr
  2021-07-25 21:09 ` JamiKettunen
  5 siblings, 0 replies; 7+ messages in thread
From: ericonr @ 2021-07-25 21:01 UTC (permalink / raw)
  To: ml

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

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

pipewire: remove libpulseaudio-pipewire from repos.
https://github.com/void-linux/void-packages/pull/32200

Description:
The packages for this are [still around on repos](https://voidlinux.org/packages/?arch=x86_64&q=libpulseaudio-pipewire), I think it's about time for those to go.
This subpackage [was deprecated with the upgrade to PipeWire 0.3.18](https://github.com/void-linux/void-packages/pull/27279).

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] 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
-->


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

* Re: pipewire: remove libpulseaudio-pipewire from repos.
  2021-07-25 20:33 [PR PATCH] pipewire: remove libpulseaudio-pipewire from repos JamiKettunen
                   ` (4 preceding siblings ...)
  2021-07-25 21:01 ` [PR PATCH] [Closed]: " ericonr
@ 2021-07-25 21:09 ` JamiKettunen
  5 siblings, 0 replies; 7+ messages in thread
From: JamiKettunen @ 2021-07-25 21:09 UTC (permalink / raw)
  To: ml

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

New comment by JamiKettunen on void-packages repository

https://github.com/void-linux/void-packages/pull/32200#issuecomment-886258060

Comment:
Ah, I didn't exactly know what should've been done in this case. Seems it was also [already reported on that issue](https://github.com/void-linux/void-packages/issues/23575#issuecomment-790695753) :)

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

end of thread, other threads:[~2021-07-25 21:09 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-25 20:33 [PR PATCH] pipewire: remove libpulseaudio-pipewire from repos JamiKettunen
2021-07-25 20:40 ` [PR PATCH] [Updated] " JamiKettunen
2021-07-25 20:41 ` JamiKettunen
2021-07-25 20:50 ` paper42
2021-07-25 21:01 ` ericonr
2021-07-25 21:01 ` [PR PATCH] [Closed]: " ericonr
2021-07-25 21:09 ` JamiKettunen

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).