* [PR PATCH] volctl: fix dependencies
@ 2021-01-12 22:54 endigma-4426
2021-01-13 2:02 ` ericonr
` (7 more replies)
0 siblings, 8 replies; 9+ messages in thread
From: endigma-4426 @ 2021-01-12 22:54 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 495 bytes --]
There is a new pull request by endigma-4426 against master on the void-packages repository
https://github.com/endigma-4426/void-packages patch-4
https://github.com/void-linux/void-packages/pull/27872
volctl: fix dependencies
volctl package requires pulseaudio currently, also totally functional with new audio server pipewire's built in pulse server, so the dependency on pulseaudio should be removed.
A patch file from https://github.com/void-linux/void-packages/pull/27872.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-patch-4-27872.patch --]
[-- Type: text/x-diff, Size: 1107 bytes --]
From 8b20324e5f8dd0b2734dd725a287a3dbaebb7834 Mon Sep 17 00:00:00 2001
From: endigma <34945114+endigma-4426@users.noreply.github.com>
Date: Tue, 12 Jan 2021 18:53:55 -0400
Subject: [PATCH] volctl: fix dependencies
volctl package requires pulseaudio currently, also totally functional with new audio server pipewire's built in pulse server, so the dependency on pulseaudio should be removed.
---
srcpkgs/volctl/template | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/volctl/template b/srcpkgs/volctl/template
index 305e7e9ba1f..0b4dc7d4409 100644
--- a/srcpkgs/volctl/template
+++ b/srcpkgs/volctl/template
@@ -4,8 +4,8 @@ version=0.8.0
revision=2
build_style=python3-module
hostmakedepends="python3-setuptools"
-depends="python3-gobject pulseaudio desktop-file-utils"
-short_desc="Lightweight volume control that sits in your systray"
+depends="python3-gobject desktop-file-utils"
+short_desc="Lightweight volume control for pulseaudio that sits in your systray"
maintainer="teldra <teldra@rotce.de>"
license="GPL-2.0-only"
homepage="https://buzz.github.io/volctl/"
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: volctl: fix dependencies
2021-01-12 22:54 [PR PATCH] volctl: fix dependencies endigma-4426
@ 2021-01-13 2:02 ` ericonr
2021-01-13 4:14 ` endigma-4426
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2021-01-13 2:02 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 369 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/27872#issuecomment-759153790
Comment:
Can you try to fix the tests? Might be a matter of adding `python3-yaml` to `depends` (also anything missing from `install_requires` line in https://github.com/buzz/volctl/blob/master/setup.py) + adding `checkdepends="$depends"`
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: volctl: fix dependencies
2021-01-12 22:54 [PR PATCH] volctl: fix dependencies endigma-4426
2021-01-13 2:02 ` ericonr
@ 2021-01-13 4:14 ` endigma-4426
2021-01-17 11:44 ` teldra
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: endigma-4426 @ 2021-01-13 4:14 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 250 bytes --]
New comment by endigma-4426 on void-packages repository
https://github.com/void-linux/void-packages/pull/27872#issuecomment-759193941
Comment:
I'll look into fixing these tomorrow, currently installing with pulseaudio on an xbps ignore list works.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: volctl: fix dependencies
2021-01-12 22:54 [PR PATCH] volctl: fix dependencies endigma-4426
2021-01-13 2:02 ` ericonr
2021-01-13 4:14 ` endigma-4426
@ 2021-01-17 11:44 ` teldra
2021-01-18 12:55 ` endigma-4426
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: teldra @ 2021-01-17 11:44 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 602 bytes --]
New comment by teldra on void-packages repository
https://github.com/void-linux/void-packages/pull/27872#issuecomment-761795633
Comment:
RIght now, I have this https://github.com/teldra/void-packages/pull/new/update-volctl
Test stops with
```
Unable to init server: Could not connect: Connection refused
Unable to init server: Could not connect: Connection refused
error: libpulse.so.0: cannot open shared object file: No such file or directory
```
So which libpulse.so.0 should be put into checkdepends?
And I wonder, if gtk+3 is really needed in depends. In checkdepends, it is needed.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: volctl: fix dependencies
2021-01-12 22:54 [PR PATCH] volctl: fix dependencies endigma-4426
` (2 preceding siblings ...)
2021-01-17 11:44 ` teldra
@ 2021-01-18 12:55 ` endigma-4426
2021-01-18 12:56 ` endigma-4426
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: endigma-4426 @ 2021-01-18 12:55 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 749 bytes --]
New comment by endigma-4426 on void-packages repository
https://github.com/void-linux/void-packages/pull/27872#issuecomment-762233457
Comment:
> RIght now, I have this https://github.com/teldra/void-packages/pull/new/update-volctl
>
> Test stops with
>
> ```
> Unable to init server: Could not connect: Connection refused
> Unable to init server: Could not connect: Connection refused
> error: libpulse.so.0: cannot open shared object file: No such file or directory
> ```
>
> So which libpulse.so.0 should be put into checkdepends?
> And I wonder, if gtk+3 is really needed in depends. In checkdepends, it is needed.
I suppose pipewire's pulse backend provides this, is there a way for an either-or dependency to be implemented?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: volctl: fix dependencies
2021-01-12 22:54 [PR PATCH] volctl: fix dependencies endigma-4426
` (3 preceding siblings ...)
2021-01-18 12:55 ` endigma-4426
@ 2021-01-18 12:56 ` endigma-4426
2021-01-18 13:46 ` ericonr
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: endigma-4426 @ 2021-01-18 12:56 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 831 bytes --]
New comment by endigma-4426 on void-packages repository
https://github.com/void-linux/void-packages/pull/27872#issuecomment-762233457
Comment:
> RIght now, I have this https://github.com/teldra/void-packages/pull/new/update-volctl
>
> Test stops with
>
> ```
> Unable to init server: Could not connect: Connection refused
> Unable to init server: Could not connect: Connection refused
> error: libpulse.so.0: cannot open shared object file: No such file or directory
> ```
>
> So which libpulse.so.0 should be put into checkdepends?
> And I wonder, if gtk+3 is really needed in depends. In checkdepends, it is needed.
I suppose pipewire's pulse backend provides this, is there a way for an either-or dependency to be implemented? I have volctl working as intended right now running under pipewire's pulse server
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: volctl: fix dependencies
2021-01-12 22:54 [PR PATCH] volctl: fix dependencies endigma-4426
` (4 preceding siblings ...)
2021-01-18 12:56 ` endigma-4426
@ 2021-01-18 13:46 ` ericonr
2021-01-28 5:21 ` ericonr
2021-02-15 4:13 ` [PR PATCH] [Closed]: " ericonr
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2021-01-18 13:46 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 314 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/27872#issuecomment-762261102
Comment:
Since the main usage is with pulseaudio, I'd also test with it. You might need to add `dbus` to `checkdepends` and run `dbus-run-session python setup.py test` in `do_check`.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: volctl: fix dependencies
2021-01-12 22:54 [PR PATCH] volctl: fix dependencies endigma-4426
` (5 preceding siblings ...)
2021-01-18 13:46 ` ericonr
@ 2021-01-28 5:21 ` ericonr
2021-02-15 4:13 ` [PR PATCH] [Closed]: " ericonr
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2021-01-28 5:21 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 183 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/27872#issuecomment-768806987
Comment:
You haven't revbumped the template either.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PR PATCH] [Closed]: volctl: fix dependencies
2021-01-12 22:54 [PR PATCH] volctl: fix dependencies endigma-4426
` (6 preceding siblings ...)
2021-01-28 5:21 ` ericonr
@ 2021-02-15 4:13 ` ericonr
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2021-02-15 4:13 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 334 bytes --]
There's a closed pull request on the void-packages repository
volctl: fix dependencies
https://github.com/void-linux/void-packages/pull/27872
Description:
volctl package requires pulseaudio currently, also totally functional with new audio server pipewire's built in pulse server, so the dependency on pulseaudio should be removed.
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-02-15 4:13 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-12 22:54 [PR PATCH] volctl: fix dependencies endigma-4426
2021-01-13 2:02 ` ericonr
2021-01-13 4:14 ` endigma-4426
2021-01-17 11:44 ` teldra
2021-01-18 12:55 ` endigma-4426
2021-01-18 12:56 ` endigma-4426
2021-01-18 13:46 ` ericonr
2021-01-28 5:21 ` ericonr
2021-02-15 4:13 ` [PR PATCH] [Closed]: " ericonr
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).