Github messages for voidlinux
 help / color / mirror / Atom feed
From: ahesford <ahesford@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: [PR PATCH] gst-plugins-base1: break cycle by making libvisual dependency optional
Date: Tue, 04 Feb 2025 20:10:34 +0100	[thread overview]
Message-ID: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-54218@inbox.vuxu.org> (raw)

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

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

https://github.com/ahesford/void-packages one-eyed-man
https://github.com/void-linux/void-packages/pull/54218

gst-plugins-base1: break cycle by making libvisual dependency optional
Closes: #54215

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

I have no idea if the absence of this plugin breaks any packages as we ship them, or how many people's workflows will be disrupted, but cutting `libvisual` out of the loop seems to be the easiest way to break a dependency cycle.

cc: @Piraty 

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-one-eyed-man-54218.patch --]
[-- Type: text/x-diff, Size: 2366 bytes --]

From 1a406d43197a16be9e77c4018586f64ec2067718 Mon Sep 17 00:00:00 2001
From: "Andrew J. Hesford" <ajh@sideband.org>
Date: Tue, 4 Feb 2025 14:04:37 -0500
Subject: [PATCH] gst-plugins-base1: break cycle by making libvisual dependency
 optional

Closes: #54215
---
 srcpkgs/gst-plugins-base1/template | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/gst-plugins-base1/template b/srcpkgs/gst-plugins-base1/template
index 2e56ee557cddb6..28e42911419988 100644
--- a/srcpkgs/gst-plugins-base1/template
+++ b/srcpkgs/gst-plugins-base1/template
@@ -1,18 +1,18 @@
 # Template file for 'gst-plugins-base1'
 pkgname=gst-plugins-base1
 version=1.24.10
-revision=1
+revision=2
 build_style=meson
 build_helper="gir"
 configure_args="-Dtremor=disabled -Dexamples=disabled -Ddoc=disabled
  $(vopt_feature cdparanoia cdparanoia) $(vopt_feature gir introspection)
- $(vopt_feature sndio sndio)"
+ $(vopt_feature sndio sndio) $(vopt_feature libvisual libvisual)"
 hostmakedepends="gettext pkg-config glib-devel orc $(vopt_if wayland wayland-devel)"
 makedepends="gstreamer1-devel glib-devel libxml2-devel pango-devel
  cairo-devel liboil-devel alsa-lib-devel libXv-devel libXext-devel
- libvisual-devel libgudev-devel libtheora-devel libvorbis-devel
- libSM-devel orc-devel $(vopt_if cdparanoia libcdparanoia-devel)
- opus-devel MesaLib-devel $(vopt_if sndio sndio-devel) graphene-devel
+ libgudev-devel libtheora-devel libvorbis-devel libSM-devel orc-devel
+ graphene-devel opus-devel MesaLib-devel $(vopt_if cdparanoia libcdparanoia-devel) 
+ $(vopt_if libvisual libvisual-devel) $(vopt_if sndio sndio-devel)
  $(vopt_if wayland 'wayland-devel wayland-protocols')"
 depends="orc>=0.4.18 gstreamer1>=${version}"
 checkdepends="mesa-dri"
@@ -24,8 +24,9 @@ changelog="https://gstreamer.freedesktop.org/releases/${version%.*}/#${version}"
 distfiles="https://gstreamer.freedesktop.org/src/gst-plugins-base/gst-plugins-base-${version}.tar.xz"
 checksum=ebd57b1be924c6e24f327dd55bab9d8fbaaebe5e1dc8fca784182ab2b12d23eb
 
-build_options="cdparanoia gir sndio wayland"
+build_options="cdparanoia gir libvisual sndio wayland"
 build_options_default="cdparanoia gir wayland"
+desc_option_libvisual="Enable audio visualization plugin"
 desc_option_sndio="Enable sndio support (unsupported, known to be problematic)"
 
 # Remove sndio if not upstreamed

             reply	other threads:[~2025-02-04 19:10 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-04 19:10 ahesford [this message]
2025-02-04 19:10 ` [PR PATCH] [Updated] " ahesford
2025-02-11 13:49 ` [PR PATCH] [Merged]: " ahesford
2025-02-11 13:55 ` Piraty
2025-02-11 14:04 ` hartwork
2025-02-11 14:04 ` hartwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-54218@inbox.vuxu.org \
    --to=ahesford@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).