GNU bug report logs - #74517
[PATCH 0/8] Remove gtk@4 input dependency on qtbase@5.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 74517 in the body.
You can then email your comments to 74517 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: guix-patches <at> gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 0/8] Remove gtk <at> 4 input dependency on qtbase <at> 5.
Date: Sun, 24 Nov 2024 23:09:12 +0100

I found that curious that building gtk <at> 4 inherently requires qtbase <at> 5
in the store's inputs.  Turns out it's based on a few libraries that
have optional graphical helping tools.  I've disabled them using
minimal variants.  After this patch series there are no input-only
path from gtk <at> 4 to qtbase <at> 5.

Nicolas Graves (8):
  gnu: Add v4l-utils-minimal.
  gnu: zbar: Split outputs.
  gnu: Add zbar-minimal.
  gnu: libde265: Remove unused qtbase-5 input.
  gnu: libde265: Update to 1.0.14.
  gnu: libdc1394: Replace v4l-utils by v4l-utils-minimal.
  gnu: gst-plugins-bad: Replace inputs with -minimal variants.
  gnu: gst-plugins-good: Replace v4l-utils by v4l-utils-minimal.

 gnu/packages/aidc.scm      | 119 ++++++++++++++++++++++++++++++-------
 gnu/packages/gstreamer.scm |   8 +--
 gnu/packages/video.scm     |  41 +++++++++----
 3 files changed, 132 insertions(+), 36 deletions(-)

-- 
2.46.0

Message #8 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 1/8] gnu: Add v4l-utils-minimal.
Date: Sun, 24 Nov 2024 23:29:08 +0100

* gnu/packages/video.scm (v4l-utils-minimal): New variable.
---
 gnu/packages/video.scm | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 2da458ed01..4cc6735dca 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -4035,6 +4035,30 @@ (define-public v4l-utils
     ;; libv4l2 is LGPL2.1+, while utilities are GPL2 only.
     (license (list license:lgpl2.1+ license:gpl2))))
 
+(define-public v4l-utils-minimal
+  (package
+    (inherit v4l-utils)
+    (source #f)
+    (name "v4l-utils-minimal")
+    (build-system trivial-build-system)
+    (arguments
+     (list
+      #:builder
+      (with-imported-modules '((guix build utils))
+        #~(begin
+            (use-modules (guix build utils))
+            (copy-recursively #$(this-package-native-input "v4l-utils")
+                              #$output)
+            (substitute* (find-files (string-append #$output "/lib/pkgconfig")
+                                     "\\.pc$")
+              (("^prefix=.*")
+               (string-append "prefix=" #$output "\n")))))))
+    (outputs '("out"))
+    (native-inputs (modify-inputs (package-native-inputs v4l-utils)
+                     (prepend v4l-utils)))
+    (inputs (modify-inputs (package-inputs v4l-utils)
+              (delete "qtbase")))))
+
 (define-public obs
   (package
     (name "obs")
-- 
2.46.0

Message #11 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 2/8] gnu: zbar: Split outputs.
Date: Sun, 24 Nov 2024 23:29:09 +0100

* gnu/packages/aidc.scm (zbar): Split outputs.
[build-system]: Switch to glib-or-gtk-build-system.
[arguments]<#:configure-flags>: Add --disable-static.
<#:modules>: Use srfi-26 and gremlin's file-runpath.
<#:phases>: Add phase 'split-outputs.
[native-inputs]: Improve style.
[inputs]: Improve style. Replace v4l-utils by v4l-utils-minimal.
---
 gnu/packages/aidc.scm | 94 +++++++++++++++++++++++++++++++++----------
 1 file changed, 73 insertions(+), 21 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index bc9c47a480..281f87dc78 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -5,6 +5,7 @@
 ;;; Copyright © 2018, 2019, 2022 Tobias Geerinckx-Rice <me <at> tobias.gr>
 ;;; Copyright © 2019 Guillaume Le Vaillant <glv <at> posteo.net>
 ;;; Copyright © 2020 Leo Famulari <leo <at> famulari.name>
+;;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -30,6 +31,7 @@ (define-module (gnu packages aidc)
   #:use-module (guix git-download)
   #:use-module (gnu packages autotools)
   #:use-module (gnu packages check)
+  #:use-module (gnu packages elf)
   #:use-module (gnu packages imagemagick)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages glib)
@@ -43,7 +45,8 @@ (define-module (gnu packages aidc)
   #:use-module (gnu packages qt)
   #:use-module (gnu packages video)
   #:use-module (guix build-system cmake)
-  #:use-module (guix build-system gnu))
+  #:use-module (guix build-system gnu)
+  #:use-module (guix build-system glib-or-gtk))
 
 (define-public zxing-cpp
   ;; Use the master branch as it includes unreleased build system improvements
@@ -184,32 +187,81 @@ (define-public zbar
          (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32
-         "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
-    (build-system gnu-build-system)
+        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
+    (build-system glib-or-gtk-build-system)
+    ;; XXX: qt output is broken: zbarcam-qt fails with segmentation fault, but
+    ;; this error happened before the output split.
+    (outputs '("out" "gtk" "qt"))
     (arguments
-     '(#:configure-flags (list "--with-gtk=auto"
+     (list
+      #:configure-flags '(list "--disable-static"
+                               "--with-gtk=auto"
                                "--with-python=auto"
                                (string-append "--with-dbusconfdir="
                                               (assoc-ref %outputs "out")
-                                              "/etc"))))
+                                              "/etc"))
+      #:modules `(((guix build gremlin) #:select (file-runpath))
+                  (srfi srfi-26)
+                  ,@%glib-or-gtk-build-system-modules)
+      #:phases
+      #~(modify-phases %standard-phases
+          (add-after 'install 'split-outputs
+            (lambda* (#:key outputs #:allow-other-keys)
+              (let ((out (assoc-ref outputs "out"))
+                    (gtk (assoc-ref outputs "gtk")))
+
+                ;; XXX: Using set-file-runpath leads to runpath-too-long-error
+                (define (prepend-to-runpath path file)
+                  (invoke "patchelf" "--set-rpath"
+                          (string-join (cons* path (file-runpath file)) ":")
+                          file))
+
+                (define (move-output-file target file)
+                  (let* ((source (string-append out file))
+                         (destination (string-append target file)))
+                    (mkdir-p (dirname destination))
+                    (rename-file source destination)
+                    (when (and (eq? (stat:type (lstat destination)) 'regular)
+                               (elf-file? destination))
+                      (prepend-to-runpath
+                       (string-append out "/lib:" target "/lib")
+                       destination))))
+
+                (for-each
+                 (lambda (kind)
+                   (for-each
+                    (cut move-output-file (assoc-ref outputs kind) <>)
+                    (cons*
+                     (string-append "/lib/pkgconfig/zbar-" kind ".pc")
+                     (string-append "/bin/zbarcam-" kind)
+                     (map (cut string-drop <> 1)
+                          (with-directory-excursion out
+                            (find-files
+                             "./lib"
+                             (string-append "lib.*" kind "\\.so.*")))))))
+                 '("gtk" "qt"))
+
+                (for-each
+                 (cut move-output-file gtk <>)
+                 '("/share/gir-1.0" "/lib/girepository-1.0"))))))))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("gettext" ,gettext-minimal)
-       ("glib" ,glib "bin")
-       ("gobject-introspection" ,gobject-introspection)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python-wrapper" ,python-wrapper)))
+     (list autoconf
+           automake
+           gettext-minimal
+           `(,glib "bin")
+           gobject-introspection
+           libtool
+           patchelf
+           pkg-config
+           python-wrapper))
     (inputs
-     `(("dbus" ,dbus)
-       ("imagemagick" ,imagemagick)
-       ("libjpeg" ,libjpeg-turbo)
-       ("perl" ,perl)
-       ("python" ,python)
-       ("qtx11extras" ,qtx11extras)
-       ("v4l-utils" ,v4l-utils)))
+     (list dbus
+           imagemagick
+           libjpeg-turbo
+           perl
+           python
+           qtx11extras
+           v4l-utils-minimal))
     (propagated-inputs
      ;; These are in 'requires' field of .pc files.
      (list glib gtk+ qtbase-5))
-- 
2.46.0

Message #14 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 3/8] gnu: Add zbar-minimal.
Date: Sun, 24 Nov 2024 23:29:10 +0100

* gnu/packages/aidc.scm (zbar-minimal): New variable.
---
 gnu/packages/aidc.scm | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index 281f87dc78..58b5b9a6f6 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -46,7 +46,8 @@ (define-module (gnu packages aidc)
   #:use-module (gnu packages video)
   #:use-module (guix build-system cmake)
   #:use-module (guix build-system gnu)
-  #:use-module (guix build-system glib-or-gtk))
+  #:use-module (guix build-system glib-or-gtk)
+  #:use-module (guix build-system trivial))
 
 (define-public zxing-cpp
   ;; Use the master branch as it includes unreleased build system improvements
@@ -277,6 +278,30 @@ (define (move-output-file target file)
     (home-page "https://github.com/mchehab/zbar")
     (license license:lgpl2.1+)))
 
+(define-public zbar-minimal
+  (package
+    (inherit zbar)
+    (source #f)
+    (name "zbar-minimal")
+    (build-system trivial-build-system)
+    (arguments
+     (list
+      #:builder
+      (with-imported-modules '((guix build utils))
+        #~(begin
+            (use-modules (guix build utils))
+            (copy-recursively #$(this-package-native-input "zbar")
+                              #$output)
+            (substitute* (string-append #$output "/lib/pkgconfig/zbar.pc")
+              (("^prefix=.*")
+               (string-append "prefix=" #$output "\n")))))))
+    (outputs '("out"))
+    (native-inputs (modify-inputs (package-native-inputs zbar)
+                     (prepend zbar)))
+    (inputs (modify-inputs (package-inputs zbar)
+              (delete "qtx11extras")))
+    (propagated-inputs '())))
+
 (define-public qrcodegen-cpp
   (package
     (name "qrcodegen-cpp")
-- 
2.46.0

Message #17 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 5/8] gnu: libde265: Update to 1.0.14.
Date: Sun, 24 Nov 2024 23:29:12 +0100

* gnu/packages/video.scm (libde265): Update to 1.0.14.
[inputs]: Replace sdl by sdl2.
---
 gnu/packages/video.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 8dd647bf20..84e4a4fe3a 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -695,7 +695,7 @@ (define-public libvideogfx
 (define-public libde265
   (package
     (name "libde265")
-    (version "1.0.8")
+    (version "1.0.14")
     (source
      (origin
        (method git-fetch)
@@ -705,7 +705,7 @@ (define-public libde265
          (commit (string-append "v" version))))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "1dzflqbk248lz5ws0ni5acmf32b3rmnq5gsfaz7691qqjxkl1zml"))))
+        (base32 "1a4n1hnr9ybji87irg2kya95slf4jnybnkg4x2zihnqqihbnv539"))))
     (build-system gnu-build-system)
     (arguments
      `(#:configure-flags
@@ -715,7 +715,7 @@ (define-public libde265
     (inputs
      ;; XXX: Build a complete version using libswscale or libvideogfx
      ;; and qtbase-5.
-     (list sdl))
+     (list sdl2))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.46.0

Message #20 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 4/8] gnu: libde265: Remove unused qtbase-5 input.
Date: Sun, 24 Nov 2024 23:29:11 +0100

* gnu/packages/video.scm (libde265)
[native-inputs]: Improve style.
[inputs]: Improve style.  Remove qtbase-5 which wasn't actually
used (need more inputs to build graphical tools).
---
 gnu/packages/video.scm | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 4cc6735dca..8dd647bf20 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -711,16 +711,11 @@ (define-public libde265
      `(#:configure-flags
        (list "--disable-static")))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python" ,python-wrapper)))
+     (list autoconf automake libtool pkg-config python-wrapper))
     (inputs
-     `(;; XXX: Build fails with libvideogfx.
-       ;; ("libvideogfx" ,libvideogfx)
-       ("qt" ,qtbase-5)
-       ("sdl" ,sdl)))
+     ;; XXX: Build a complete version using libswscale or libvideogfx
+     ;; and qtbase-5.
+     (list sdl))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.46.0

Message #23 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 6/8] gnu: libdc1394: Replace v4l-utils by v4l-utils-minimal.
Date: Sun, 24 Nov 2024 23:29:13 +0100

* gnu/packages/gstreamer.scm (libdc1394)[inputs]: Replace v4l-utils by
v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 13c8734e68..5ce33faa51 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -160,7 +160,7 @@ (define-public libdc1394
            libxv
            mesa
            sdl
-           v4l-utils))
+           v4l-utils-minimal))
     (synopsis "1394-Based Digital Camera Control Library")
     (description "LibDC1394 is a library that provides functionality to control
 any camera that conforms to the 1394-Based Digital Camera Specification written
-- 
2.46.0

Message #26 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 7/8] gnu: gst-plugins-bad: Replace inputs with -minimal
 variants.
Date: Sun, 24 Nov 2024 23:29:14 +0100

* gnu/packages/gstreamer.scm (gst-plugins-bad)[inputs]: Replace
v4l-utils by v4l-utils-minimal. Replace zbar by zbar-minimal.
---
 gnu/packages/gstreamer.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 5ce33faa51..bb1bdcb4a1 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -936,7 +936,7 @@ (define-public gst-plugins-bad
             tinyalsa
             transcode
             usrsctp
-            v4l-utils
+            v4l-utils-minimal
             vo-aacenc
             vo-amrwbenc
             vulkan-headers
@@ -946,7 +946,7 @@ (define-public gst-plugins-bad
             webrtc-audio-processing
             wildmidi
             wpebackend-fdo
-            zbar
+            zbar-minimal
             zxing-cpp-1.2)))
     (home-page "https://gstreamer.freedesktop.org/")
     (synopsis "Plugins for the GStreamer multimedia library")
-- 
2.46.0

Message #29 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 8/8] gnu: gst-plugins-good: Replace v4l-utils by
 v4l-utils-minimal.
Date: Sun, 24 Nov 2024 23:29:15 +0100

* gnu/packages/gstreamer.scm (gst-plugins-good)[inputs]: Replace
v4l-utils by v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index bb1bdcb4a1..905ac2d79a 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -732,7 +732,7 @@ (define libsoup
            speex
            taglib
            twolame
-           v4l-utils
+           v4l-utils-minimal
            wavpack
            zlib))
     (propagated-inputs
-- 
2.46.0

Message #32 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 0/8] Remove gtk <at> 4 input dependency on qtbase <at> 5.
Date: Mon, 25 Nov 2024 11:38:08 +0900

Hi Nicolas,

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> I found that curious that building gtk <at> 4 inherently requires qtbase <at> 5
> in the store's inputs.  Turns out it's based on a few libraries that
> have optional graphical helping tools.  I've disabled them using
> minimal variants.  After this patch series there are no input-only
> path from gtk <at> 4 to qtbase <at> 5.

I agree that was sub-optimal, thank you for addressing that.  This
actually would resolve #51994 I had reported some time ago ("GTK 4
requires qtbase through various sources (gstreamer plugins)"), so you
(or the person who pushes) should add this as a git trailer:

--8<---------------cut here---------------start------------->8---
Fixes: https://issues.guix.gnu.org/51994
--8<---------------cut here---------------end--------------->8---

and close the associate bug after it's pushed.

-- 
Thanks,
Maxim

Message #35 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 1/8] gnu: Add v4l-utils-minimal.
Date: Mon, 25 Nov 2024 11:54:19 +0900

Hi,

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> * gnu/packages/video.scm (v4l-utils-minimal): New variable.
> ---
>  gnu/packages/video.scm | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index 2da458ed01..4cc6735dca 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -4035,6 +4035,30 @@ (define-public v4l-utils
>      ;; libv4l2 is LGPL2.1+, while utilities are GPL2 only.
>      (license (list license:lgpl2.1+ license:gpl2))))
>  
> +(define-public v4l-utils-minimal
> +  (package
> +    (inherit v4l-utils)
> +    (source #f)
> +    (name "v4l-utils-minimal")
> +    (build-system trivial-build-system)
> +    (arguments
> +     (list
> +      #:builder
> +      (with-imported-modules '((guix build utils))
> +        #~(begin
> +            (use-modules (guix build utils))
> +            (copy-recursively #$(this-package-native-input "v4l-utils")
> +                              #$output)
> +            (substitute* (find-files (string-append #$output "/lib/pkgconfig")
> +                                     "\\.pc$")
> +              (("^prefix=.*")
> +               (string-append "prefix=" #$output "\n")))))))
> +    (outputs '("out"))

I don't understand how this resolves the issue, at a glance.  Is the
issue that some v4l-utils tools, kept in a distinct output, are referred
to in the pkg-config files, present in the main output?

A comment explaining would be beneficial, but mostly I would rather we
use the more traditional style of rebuilding the package with the
modified inputs, so that it's cleanly built as a 'qtbase-less' variant;
that should lead to less bad surprises down the road, at the expense of
a bit more computation, which we can afford.

As a variant (a package sharing the same source but using different
inputs/build flags), the package should be defined via
'package/inherit', which will honor the parent package's replacements
(grafts).

-- 
Thanks,
Maxim

Message #38 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 1/8] gnu: Add v4l-utils-minimal.
Date: Mon, 25 Nov 2024 11:56:17 +0900

Also,

> * gnu/packages/video.scm (v4l-utils-minimal): New variable.

I'd explain in a comment what this variant is for (break dependency on
Qt); better yet, a #:disallowed-reference could be use along such
comment to ensure the property is preserved in time.

Thanks.

Message #41 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 2/8] gnu: zbar: Split outputs.
Date: Mon, 25 Nov 2024 12:04:39 +0900

Hi,

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> * gnu/packages/aidc.scm (zbar): Split outputs.
> [build-system]: Switch to glib-or-gtk-build-system.
> [arguments]<#:configure-flags>: Add --disable-static.
> <#:modules>: Use srfi-26 and gremlin's file-runpath.
> <#:phases>: Add phase 'split-outputs.
> [native-inputs]: Improve style.
> [inputs]: Improve style. Replace v4l-utils by v4l-utils-minimal.
> ---
>  gnu/packages/aidc.scm | 94 +++++++++++++++++++++++++++++++++----------
>  1 file changed, 73 insertions(+), 21 deletions(-)

[...]

>  (define-public zxing-cpp
>    ;; Use the master branch as it includes unreleased build system improvements
> @@ -184,32 +187,81 @@ (define-public zbar
>           (commit version)))
>         (file-name (git-file-name name version))
>         (sha256
> -        (base32
> -         "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
> -    (build-system gnu-build-system)
> +        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
> +    (build-system glib-or-gtk-build-system)
> +    ;; XXX: qt output is broken: zbarcam-qt fails with segmentation fault, but
> +    ;; this error happened before the output split.

As an alternative to packaging broken outputs, perhaps we could simply
tell the build system to not build with Qt, or wholly remove it from the
inputs?

> +    (outputs '("out" "gtk" "qt"))
>      (arguments
> -     '(#:configure-flags (list "--with-gtk=auto"
> +     (list
> +      #:configure-flags '(list "--disable-static"
> +                               "--with-gtk=auto"
>                                 "--with-python=auto"
>                                 (string-append "--with-dbusconfdir="
>                                                (assoc-ref %outputs "out")
> -                                              "/etc"))))
> +                                              "/etc"))
> +      #:modules `(((guix build gremlin) #:select (file-runpath))
> +                  (srfi srfi-26)
> +                  ,@%glib-or-gtk-build-system-modules)
> +      #:phases
> +      #~(modify-phases %standard-phases
> +          (add-after 'install 'split-outputs
> +            (lambda* (#:key outputs #:allow-other-keys)
> +              (let ((out (assoc-ref outputs "out"))
> +                    (gtk (assoc-ref outputs "gtk")))

This can be rewritten using the gexp variabls #$output and #$output:gtk.
> +
> +                ;; XXX: Using set-file-runpath leads to runpath-too-long-error
> +                (define (prepend-to-runpath path file)
> +                  (invoke "patchelf" "--set-rpath"
> +                          (string-join (cons* path (file-runpath file)) ":")
> +                          file))
> +
> +                (define (move-output-file target file)
> +                  (let* ((source (string-append out file))
> +                         (destination (string-append target file)))
> +                    (mkdir-p (dirname destination))
> +                    (rename-file source destination)
> +                    (when (and (eq? (stat:type (lstat destination)) 'regular)
> +                               (elf-file? destination))
> +                      (prepend-to-runpath
> +                       (string-append out "/lib:" target "/lib")
> +                       destination))))
> +
> +                (for-each
> +                 (lambda (kind)
> +                   (for-each
> +                    (cut move-output-file (assoc-ref outputs kind) <>)
> +                    (cons*
> +                     (string-append "/lib/pkgconfig/zbar-" kind ".pc")
> +                     (string-append "/bin/zbarcam-" kind)
> +                     (map (cut string-drop <> 1)
> +                          (with-directory-excursion out
> +                            (find-files
> +                             "./lib"
> +                             (string-append "lib.*" kind "\\.so.*")))))))
> +                 '("gtk" "qt"))
> +
> +                (for-each
> +                 (cut move-output-file gtk <>)
> +                 '("/share/gir-1.0" "/lib/girepository-1.0"))))))))

That is clever, perhaps too clever (in comparison to the simple brute
force approach of using a distinct minimal package variant built without
the extra inputs) :-).

>      (native-inputs
> -     `(("autoconf" ,autoconf)
> -       ("automake" ,automake)
> -       ("gettext" ,gettext-minimal)
> -       ("glib" ,glib "bin")
> -       ("gobject-introspection" ,gobject-introspection)
> -       ("libtool" ,libtool)
> -       ("pkg-config" ,pkg-config)
> -       ("python-wrapper" ,python-wrapper)))
> +     (list autoconf
> +           automake
> +           gettext-minimal
> +           `(,glib "bin")
> +           gobject-introspection
> +           libtool
> +           patchelf
> +           pkg-config
> +           python-wrapper))
>      (inputs
> -     `(("dbus" ,dbus)
> -       ("imagemagick" ,imagemagick)
> -       ("libjpeg" ,libjpeg-turbo)
> -       ("perl" ,perl)
> -       ("python" ,python)
> -       ("qtx11extras" ,qtx11extras)
> -       ("v4l-utils" ,v4l-utils)))
> +     (list dbus
> +           imagemagick
> +           libjpeg-turbo
> +           perl
> +           python
> +           qtx11extras
> +           v4l-utils-minimal))

Could be done in a separate commit; this entangles concern and makes it
easier if in the future a revert needs to be made.

-- 
Thanks,
Maxim

Message #44 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 3/8] gnu: Add zbar-minimal.
Date: Mon, 25 Nov 2024 12:08:12 +0900

Hi Nicolas,

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> * gnu/packages/aidc.scm (zbar-minimal): New variable.
> ---
>  gnu/packages/aidc.scm | 27 ++++++++++++++++++++++++++-
>  1 file changed, 26 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
> index 281f87dc78..58b5b9a6f6 100644
> --- a/gnu/packages/aidc.scm
> +++ b/gnu/packages/aidc.scm
> @@ -46,7 +46,8 @@ (define-module (gnu packages aidc)
>    #:use-module (gnu packages video)
>    #:use-module (guix build-system cmake)
>    #:use-module (guix build-system gnu)
> -  #:use-module (guix build-system glib-or-gtk))
> +  #:use-module (guix build-system glib-or-gtk)
> +  #:use-module (guix build-system trivial))
>  
>  (define-public zxing-cpp
>    ;; Use the master branch as it includes unreleased build system improvements
> @@ -277,6 +278,30 @@ (define (move-output-file target file)
>      (home-page "https://github.com/mchehab/zbar")
>      (license license:lgpl2.1+)))
>  
> +(define-public zbar-minimal
> +  (package
> +    (inherit zbar)
> +    (source #f)
> +    (name "zbar-minimal")
> +    (build-system trivial-build-system)
> +    (arguments
> +     (list
> +      #:builder
> +      (with-imported-modules '((guix build utils))
> +        #~(begin
> +            (use-modules (guix build utils))
> +            (copy-recursively #$(this-package-native-input "zbar")
> +                              #$output)
> +            (substitute* (string-append #$output "/lib/pkgconfig/zbar.pc")
> +              (("^prefix=.*")
> +               (string-append "prefix=" #$output "\n")))))))
> +    (outputs '("out"))
> +    (native-inputs (modify-inputs (package-native-inputs zbar)
> +                     (prepend zbar)))
> +    (inputs (modify-inputs (package-inputs zbar)
> +              (delete "qtx11extras")))
> +    (propagated-inputs '())))

I see; the previous commit neatly organized dependencies on gtk/qt in
separate outputs, and this variant ensures any reference to these output
is severed, without any rebuild.  It's a researched approach, but I'm a
bit worried about long term maintenance/correctness; I feel a classic
variant rebuilt without the problematic inputs would be both easier to
understand and maintain in the long term.

Also, same earlier comments hold about package/inherit and using
#:disallowed-reference along with an explanatory comment.

-- 
Thanks,
Maxim

Message #47 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 4/8] gnu: libde265: Remove unused qtbase-5
 input.
Date: Mon, 25 Nov 2024 12:08:56 +0900

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> * gnu/packages/video.scm (libde265)
> [native-inputs]: Improve style.
> [inputs]: Improve style.  Remove qtbase-5 which wasn't actually
> used (need more inputs to build graphical tools).

Reviewed-by: Maxim Cournoyer <maxim.cournoyer <at> gmail>

-- 
Thanks,
Maxim

Message #50 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 5/8] gnu: libde265: Update to 1.0.14.
Date: Mon, 25 Nov 2024 12:09:08 +0900

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> * gnu/packages/video.scm (libde265): Update to 1.0.14.
> [inputs]: Replace sdl by sdl2.

Reviewed-by: Maxim Cournoyer <maxim.cournoyer <at> gmail>

-- 
Thanks,
Maxim

Message #53 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 0/8] Remove gtk <at> 4 input dependency on qtbase <at> 5.
Date: Mon, 25 Nov 2024 12:11:28 +0900

Hi,

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> I found that curious that building gtk <at> 4 inherently requires qtbase <at> 5
> in the store's inputs.  Turns out it's based on a few libraries that
> have optional graphical helping tools.  I've disabled them using
> minimal variants.  After this patch series there are no input-only
> path from gtk <at> 4 to qtbase <at> 5.
>
> Nicolas Graves (8):
>   gnu: Add v4l-utils-minimal.
>   gnu: zbar: Split outputs.
>   gnu: Add zbar-minimal.
>   gnu: libde265: Remove unused qtbase-5 input.

I left some comments; I think I'd prefer a simpler rebuilt variant
solution compared to the fancier but more frail (because of more
complex) runpath hacking/trivial-build-system copying business involved
in this version.

>   gnu: libde265: Update to 1.0.14.
>   gnu: libdc1394: Replace v4l-utils by v4l-utils-minimal.
>   gnu: gst-plugins-bad: Replace inputs with -minimal variants.
>   gnu: gst-plugins-good: Replace v4l-utils by v4l-utils-minimal.

These LGTM.

Thanks a lot for tackling this problem!

-- 
Maxim

Message #56 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 2/8] gnu: zbar: Split outputs.
Date: Mon, 25 Nov 2024 08:42:43 +0100

On 2024-11-25 12:04, Maxim Cournoyer wrote:

> Hi,
>
> Nicolas Graves <ngraves <at> ngraves.fr> writes:
>
>> * gnu/packages/aidc.scm (zbar): Split outputs.
>> [build-system]: Switch to glib-or-gtk-build-system.
>> [arguments]<#:configure-flags>: Add --disable-static.
>> <#:modules>: Use srfi-26 and gremlin's file-runpath.
>> <#:phases>: Add phase 'split-outputs.
>> [native-inputs]: Improve style.
>> [inputs]: Improve style. Replace v4l-utils by v4l-utils-minimal.
>> ---
>>  gnu/packages/aidc.scm | 94 +++++++++++++++++++++++++++++++++----------
>>  1 file changed, 73 insertions(+), 21 deletions(-)
>
> [...]
>
>>  (define-public zxing-cpp
>>    ;; Use the master branch as it includes unreleased build system improvements
>> @@ -184,32 +187,81 @@ (define-public zbar
>>           (commit version)))
>>         (file-name (git-file-name name version))
>>         (sha256
>> -        (base32
>> -         "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
>> -    (build-system gnu-build-system)
>> +        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
>> +    (build-system glib-or-gtk-build-system)
>> +    ;; XXX: qt output is broken: zbarcam-qt fails with segmentation fault, but
>> +    ;; this error happened before the output split.
>
> As an alternative to packaging broken outputs, perhaps we could simply
> tell the build system to not build with Qt, or wholly remove it from the
> inputs?
>
>> +    (outputs '("out" "gtk" "qt"))
>>      (arguments
>> -     '(#:configure-flags (list "--with-gtk=auto"
>> +     (list
>> +      #:configure-flags '(list "--disable-static"
>> +                               "--with-gtk=auto"
>>                                 "--with-python=auto"
>>                                 (string-append "--with-dbusconfdir="
>>                                                (assoc-ref %outputs "out")
>> -                                              "/etc"))))
>> +                                              "/etc"))
>> +      #:modules `(((guix build gremlin) #:select (file-runpath))
>> +                  (srfi srfi-26)
>> +                  ,@%glib-or-gtk-build-system-modules)
>> +      #:phases
>> +      #~(modify-phases %standard-phases
>> +          (add-after 'install 'split-outputs
>> +            (lambda* (#:key outputs #:allow-other-keys)
>> +              (let ((out (assoc-ref outputs "out"))
>> +                    (gtk (assoc-ref outputs "gtk")))
>
> This can be rewritten using the gexp variabls #$output and #$output:gtk.
>> +
>> +                ;; XXX: Using set-file-runpath leads to runpath-too-long-error
>> +                (define (prepend-to-runpath path file)
>> +                  (invoke "patchelf" "--set-rpath"
>> +                          (string-join (cons* path (file-runpath file)) ":")
>> +                          file))
>> +
>> +                (define (move-output-file target file)
>> +                  (let* ((source (string-append out file))
>> +                         (destination (string-append target file)))
>> +                    (mkdir-p (dirname destination))
>> +                    (rename-file source destination)
>> +                    (when (and (eq? (stat:type (lstat destination)) 'regular)
>> +                               (elf-file? destination))
>> +                      (prepend-to-runpath
>> +                       (string-append out "/lib:" target "/lib")
>> +                       destination))))
>> +
>> +                (for-each
>> +                 (lambda (kind)
>> +                   (for-each
>> +                    (cut move-output-file (assoc-ref outputs kind) <>)
>> +                    (cons*
>> +                     (string-append "/lib/pkgconfig/zbar-" kind ".pc")
>> +                     (string-append "/bin/zbarcam-" kind)
>> +                     (map (cut string-drop <> 1)
>> +                          (with-directory-excursion out
>> +                            (find-files
>> +                             "./lib"
>> +                             (string-append "lib.*" kind "\\.so.*")))))))
>> +                 '("gtk" "qt"))
>> +
>> +                (for-each
>> +                 (cut move-output-file gtk <>)
>> +                 '("/share/gir-1.0" "/lib/girepository-1.0"))))))))
>
> That is clever, perhaps too clever (in comparison to the simple brute
> force approach of using a distinct minimal package variant built without
> the extra inputs) :-).

Yes, but I think the output split is desirable independently of the
minimal variant (for when we won't have to carry the propagated-inputs
anymore), and thus it makes sense to keep it.  WDYT?

I could also try to package :
- one minimal version
- one gtk version in its own package, trying not to rebuild but to link
if not too complex
- not try to package a qt version (I'm not able to debug this).

-- 
Best regards,
Nicolas Graves

Message #59 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH 0/8] Remove gtk <at> 4 input dependency on qtbase <at> 5.
Date: Mon, 25 Nov 2024 08:47:25 +0100

On 2024-11-25 12:11, Maxim Cournoyer wrote:

> Hi,
>
> Nicolas Graves <ngraves <at> ngraves.fr> writes:
>
>> I found that curious that gtk <at> 4 inherently requires qtbase <at> 5
>> in the store's inputs.  Turns out it's based on a few libraries that
>> have optional graphical helping tools.  I've disabled them using
>> minimal variants.  After this patch series there are no input-only
>> path from gtk <at> 4 to qtbase <at> 5.
>>
>> Nicolas Graves (8):
>>   gnu: Add v4l-utils-minimal.
>>   gnu: zbar: Split outputs.
>>   gnu: Add zbar-minimal.
>>   gnu: libde265: Remove unused qtbase-5 input.
>
> I left some comments; I think I'd prefer a simpler rebuilt variant
> solution compared to the fancier but more frail (because of more
> complex) runpath hacking/trivial-build-system copying business involved
> in this version.

I get it, I chose that because of its lack of need to rebuild, while the
simpler rebuild variant would indeed require a rebuild (here a copy).
It's not that big of a subject here, these libraries are not super heavy
themselves so I'll change that.

I still like this solution for this reason, and I think this could be
applied for some other packages I've worked on (e.g. libreoffice is very
tricky to separate the build into steps, because some of its build
elements require the whole package, but it can be split at the end).
Agree to keep this as an exception when the alternative is really more
complex though, or when the build is very heavy (and thus making a copy
despite additional complexity makes sense).

>>   gnu: libde265: Update to 1.0.14.
>>   gnu: libdc1394: Replace v4l-utils by v4l-utils-minimal.
>>   gnu: gst-plugins-bad: Replace inputs with -minimal variants.
>>   gnu: gst-plugins-good: Replace v4l-utils by v4l-utils-minimal.
>
> These LGTM.
>
> Thanks a lot for tackling this problem!

-- 
Best regards,
Nicolas Graves

Message #62 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 02/10] gnu: zbar: Improve style.
Date: Mon, 25 Nov 2024 10:43:51 +0100

* gnu/packages/aidc.scm (zbar)[native-inputs, inputs]: Improve style.
---
 gnu/packages/aidc.scm | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index bc9c47a480..0c1f26495b 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -194,22 +194,22 @@ (define-public zbar
                                               (assoc-ref %outputs "out")
                                               "/etc"))))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("gettext" ,gettext-minimal)
-       ("glib" ,glib "bin")
-       ("gobject-introspection" ,gobject-introspection)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python-wrapper" ,python-wrapper)))
+     (list autoconf
+           automake
+           gettext-minimal
+           `(,glib "bin")
+           gobject-introspection
+           libtool
+           patchelf
+           pkg-config
+           python-wrapper))
     (inputs
-     `(("dbus" ,dbus)
-       ("imagemagick" ,imagemagick)
-       ("libjpeg" ,libjpeg-turbo)
-       ("perl" ,perl)
-       ("python" ,python)
-       ("qtx11extras" ,qtx11extras)
-       ("v4l-utils" ,v4l-utils)))
+     (list dbus
+           imagemagick
+           libjpeg-turbo
+           perl
+           python
+           v4l-utils))
     (propagated-inputs
      ;; These are in 'requires' field of .pc files.
      (list glib gtk+ qtbase-5))
-- 
2.46.0

Message #65 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 01/10] gnu: Add v4l-utils-minimal.
Date: Mon, 25 Nov 2024 10:43:50 +0100

* gnu/packages/video.scm (v4l-utils-minimal): New variable.
---
 gnu/packages/video.scm | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 2da458ed01..cf5ff45fb7 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -4035,6 +4035,19 @@ (define-public v4l-utils
     ;; libv4l2 is LGPL2.1+, while utilities are GPL2 only.
     (license (list license:lgpl2.1+ license:gpl2))))
 
+(define-public v4l-utils-minimal
+  (package
+    (inherit v4l-utils)
+    (name "v4l-utils-minimal")
+    (arguments
+     (substitute-keyword-arguments (package-arguments v4l-utils)
+       ((#:phases phases)
+        #~(modify-phases #$phases
+            (delete 'split)))))
+    (outputs '("out"))
+    (inputs (modify-inputs (package-inputs v4l-utils)
+              (delete "qtbase")))))
+
 (define-public obs
   (package
     (name "obs")
-- 
2.46.0

Message #68 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 03/10] gnu: zbar: Disable static and qt build.
Date: Mon, 25 Nov 2024 10:43:52 +0100

* gnu/packages/aidc.scm (zbar): Disable static and qt build.
[build-system]: Switch to glib-or-gtk-build-system.
[arguments]<#:configure-flags>: Add --disable-static.
[inputs]: Replace v4l-utils by v4l-utils-minimal.
---
 gnu/packages/aidc.scm | 20 +++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index 0c1f26495b..dbbab0e22b 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -5,6 +5,7 @@
 ;;; Copyright © 2018, 2019, 2022 Tobias Geerinckx-Rice <me <at> tobias.gr>
 ;;; Copyright © 2019 Guillaume Le Vaillant <glv <at> posteo.net>
 ;;; Copyright © 2020 Leo Famulari <leo <at> famulari.name>
+;;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -28,8 +29,10 @@ (define-module (gnu packages aidc)
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix git-download)
+  #:use-module (guix utils)
   #:use-module (gnu packages autotools)
   #:use-module (gnu packages check)
+  #:use-module (gnu packages elf)
   #:use-module (gnu packages imagemagick)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages glib)
@@ -43,7 +46,8 @@ (define-module (gnu packages aidc)
   #:use-module (gnu packages qt)
   #:use-module (gnu packages video)
   #:use-module (guix build-system cmake)
-  #:use-module (guix build-system gnu))
+  #:use-module (guix build-system gnu)
+  #:use-module (guix build-system glib-or-gtk))
 
 (define-public zxing-cpp
   ;; Use the master branch as it includes unreleased build system improvements
@@ -171,6 +175,7 @@ (define-public libdmtx
 C/C++ programs to use its capabilities without restrictions or overhead.")
     (license license:bsd-3)))
 
+;; XXX: qt variant utils are broken: zbarcam-qt fails with segmentation fault.
 (define-public zbar
   (package
     (name "zbar")
@@ -184,11 +189,12 @@ (define-public zbar
          (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32
-         "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
-    (build-system gnu-build-system)
+        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
+    (build-system glib-or-gtk-build-system)
     (arguments
-     '(#:configure-flags (list "--with-gtk=auto"
+     (list
+      #:configure-flags '(list "--disable-static"
+                               "--with-gtk=auto"
                                "--with-python=auto"
                                (string-append "--with-dbusconfdir="
                                               (assoc-ref %outputs "out")
@@ -209,10 +215,10 @@ (define-public zbar
            libjpeg-turbo
            perl
            python
-           v4l-utils))
+           v4l-utils-minimal))
     (propagated-inputs
      ;; These are in 'requires' field of .pc files.
-     (list glib gtk+ qtbase-5))
+     (list glib gtk+))
     (synopsis "Bar code reader")
     (description
      "ZBar can read barcodes from various sources, such as video streams,
-- 
2.46.0

Message #71 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 04/10] gnu: zbar: Update to 0.23.92.
Date: Mon, 25 Nov 2024 10:43:53 +0100

* gnu/packages/aidc.scm (zbar): Update to 0.23.92.
---
 gnu/packages/aidc.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index dbbab0e22b..cf18801aec 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -179,7 +179,7 @@ (define-public libdmtx
 (define-public zbar
   (package
     (name "zbar")
-    (version "0.23.90")
+    (version "0.23.92")
     (source
      (origin
        (method git-fetch)
@@ -189,7 +189,7 @@ (define-public zbar
          (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
+        (base32 "0k3g0ql2m4dnflppp9r3k804d927g7zslczblzcrbvhp02g6n5an"))))
     (build-system glib-or-gtk-build-system)
     (arguments
      (list
-- 
2.46.0

Message #74 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 05/10] gnu: Add zbar-minimal.
Date: Mon, 25 Nov 2024 10:43:54 +0100

* gnu/packages/aidc.scm (zbar-minimal): New variable.
---
 gnu/packages/aidc.scm | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index cf18801aec..a08b89e252 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -231,6 +231,18 @@ (define-public zbar
     (home-page "https://github.com/mchehab/zbar")
     (license license:lgpl2.1+)))
 
+(define-public zbar-minimal
+  (package/inherit zbar
+    (name "zbar-minimal")
+    (build-system gnu-build-system)
+    (arguments
+     (substitute-keyword-arguments (package-arguments zbar)
+       ((#:configure-flags flags)
+        #~(cons* "--with-gtk=no" (delete "--with-gtk=auto" #$flags)))))
+    (propagated-inputs
+     (modify-inputs (package-propagated-inputs zbar)
+       (delete "gtk+")))))
+
 (define-public qrcodegen-cpp
   (package
     (name "qrcodegen-cpp")
-- 
2.46.0

Message #77 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 06/10] gnu: libde265: Remove unused qtbase-5 input.
Date: Mon, 25 Nov 2024 10:43:55 +0100

* gnu/packages/video.scm (libde265)
[native-inputs]: Improve style.
[inputs]: Improve style.  Remove qtbase-5 which wasn't actually
used (need more inputs to build graphical tools).
---
 gnu/packages/video.scm | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index cf5ff45fb7..2ee0058c81 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -711,16 +711,11 @@ (define-public libde265
      `(#:configure-flags
        (list "--disable-static")))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python" ,python-wrapper)))
+     (list autoconf automake libtool pkg-config python-wrapper))
     (inputs
-     `(;; XXX: Build fails with libvideogfx.
-       ;; ("libvideogfx" ,libvideogfx)
-       ("qt" ,qtbase-5)
-       ("sdl" ,sdl)))
+     ;; XXX: Build a complete version using libswscale or libvideogfx
+     ;; and qtbase-5.
+     (list sdl))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.46.0

Message #80 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 07/10] gnu: libde265: Update to 1.0.14.
Date: Mon, 25 Nov 2024 10:43:56 +0100

* gnu/packages/video.scm (libde265): Update to 1.0.14.
[inputs]: Replace sdl by sdl2.
---
 gnu/packages/video.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 2ee0058c81..6f78a966c9 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -695,7 +695,7 @@ (define-public libvideogfx
 (define-public libde265
   (package
     (name "libde265")
-    (version "1.0.8")
+    (version "1.0.14")
     (source
      (origin
        (method git-fetch)
@@ -705,7 +705,7 @@ (define-public libde265
          (commit (string-append "v" version))))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "1dzflqbk248lz5ws0ni5acmf32b3rmnq5gsfaz7691qqjxkl1zml"))))
+        (base32 "1a4n1hnr9ybji87irg2kya95slf4jnybnkg4x2zihnqqihbnv539"))))
     (build-system gnu-build-system)
     (arguments
      `(#:configure-flags
@@ -715,7 +715,7 @@ (define-public libde265
     (inputs
      ;; XXX: Build a complete version using libswscale or libvideogfx
      ;; and qtbase-5.
-     (list sdl))
+     (list sdl2))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.46.0

Message #83 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 08/10] gnu: libdc1394: Replace v4l-utils by
 v4l-utils-minimal.
Date: Mon, 25 Nov 2024 10:43:57 +0100

* gnu/packages/gstreamer.scm (libdc1394)[inputs]: Replace v4l-utils by
v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 13c8734e68..5ce33faa51 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -160,7 +160,7 @@ (define-public libdc1394
            libxv
            mesa
            sdl
-           v4l-utils))
+           v4l-utils-minimal))
     (synopsis "1394-Based Digital Camera Control Library")
     (description "LibDC1394 is a library that provides functionality to control
 any camera that conforms to the 1394-Based Digital Camera Specification written
-- 
2.46.0

Message #86 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 09/10] gnu: gst-plugins-bad: Replace inputs with -minimal
 variants.
Date: Mon, 25 Nov 2024 10:43:58 +0100

* gnu/packages/gstreamer.scm (gst-plugins-bad)[inputs]: Replace
v4l-utils by v4l-utils-minimal. Replace zbar by zbar-minimal.
---
 gnu/packages/gstreamer.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 5ce33faa51..bb1bdcb4a1 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -936,7 +936,7 @@ (define-public gst-plugins-bad
             tinyalsa
             transcode
             usrsctp
-            v4l-utils
+            v4l-utils-minimal
             vo-aacenc
             vo-amrwbenc
             vulkan-headers
@@ -946,7 +946,7 @@ (define-public gst-plugins-bad
             webrtc-audio-processing
             wildmidi
             wpebackend-fdo
-            zbar
+            zbar-minimal
             zxing-cpp-1.2)))
     (home-page "https://gstreamer.freedesktop.org/")
     (synopsis "Plugins for the GStreamer multimedia library")
-- 
2.46.0

Message #89 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 10/10] gnu: gst-plugins-good: Replace v4l-utils by
 v4l-utils-minimal.
Date: Mon, 25 Nov 2024 10:43:59 +0100

* gnu/packages/gstreamer.scm (gst-plugins-good)[inputs]: Replace
v4l-utils by v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index bb1bdcb4a1..905ac2d79a 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -732,7 +732,7 @@ (define libsoup
            speex
            taglib
            twolame
-           v4l-utils
+           v4l-utils-minimal
            wavpack
            zlib))
     (propagated-inputs
-- 
2.46.0

Message #92 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 01/10] gnu: Add v4l-utils-minimal.
Date: Mon, 25 Nov 2024 10:52:34 +0100

* gnu/packages/video.scm (v4l-utils-minimal): New variable.
---
 gnu/packages/video.scm | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 2da458ed01..6f9eea6fc2 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -4035,6 +4035,20 @@ (define-public v4l-utils
     ;; libv4l2 is LGPL2.1+, while utilities are GPL2 only.
     (license (list license:lgpl2.1+ license:gpl2))))
 
+(define-public v4l-utils-minimal
+  (package/inherit v4l-utils
+    (name "v4l-utils-minimal")
+    (arguments
+     (substitute-keyword-arguments (package-arguments v4l-utils)
+       ((#:phases phases)
+        #~(modify-phases #$phases
+            (delete 'split)))
+       ((#:disallowed-references _)
+        #~("qtbase"))))
+    (outputs '("out"))
+    (inputs (modify-inputs (package-inputs v4l-utils)
+              (delete "qtbase")))))
+
 (define-public obs
   (package
     (name "obs")
-- 
2.46.0

Message #95 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 02/10] gnu: zbar: Improve style.
Date: Mon, 25 Nov 2024 10:52:35 +0100

* gnu/packages/aidc.scm (zbar)[native-inputs, inputs]: Improve style.
---
 gnu/packages/aidc.scm | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index bc9c47a480..0c1f26495b 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -194,22 +194,22 @@ (define-public zbar
                                               (assoc-ref %outputs "out")
                                               "/etc"))))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("gettext" ,gettext-minimal)
-       ("glib" ,glib "bin")
-       ("gobject-introspection" ,gobject-introspection)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python-wrapper" ,python-wrapper)))
+     (list autoconf
+           automake
+           gettext-minimal
+           `(,glib "bin")
+           gobject-introspection
+           libtool
+           patchelf
+           pkg-config
+           python-wrapper))
     (inputs
-     `(("dbus" ,dbus)
-       ("imagemagick" ,imagemagick)
-       ("libjpeg" ,libjpeg-turbo)
-       ("perl" ,perl)
-       ("python" ,python)
-       ("qtx11extras" ,qtx11extras)
-       ("v4l-utils" ,v4l-utils)))
+     (list dbus
+           imagemagick
+           libjpeg-turbo
+           perl
+           python
+           v4l-utils))
     (propagated-inputs
      ;; These are in 'requires' field of .pc files.
      (list glib gtk+ qtbase-5))
-- 
2.46.0

Message #98 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 03/10] gnu: zbar: Disable static and qt build.
Date: Mon, 25 Nov 2024 10:52:36 +0100

* gnu/packages/aidc.scm (zbar): Disable static and qt build.
[build-system]: Switch to glib-or-gtk-build-system.
[arguments]<#:configure-flags>: Add --disable-static.
[inputs]: Replace v4l-utils by v4l-utils-minimal.
---
 gnu/packages/aidc.scm | 20 +++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index 0c1f26495b..dbbab0e22b 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -5,6 +5,7 @@
 ;;; Copyright © 2018, 2019, 2022 Tobias Geerinckx-Rice <me <at> tobias.gr>
 ;;; Copyright © 2019 Guillaume Le Vaillant <glv <at> posteo.net>
 ;;; Copyright © 2020 Leo Famulari <leo <at> famulari.name>
+;;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -28,8 +29,10 @@ (define-module (gnu packages aidc)
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix git-download)
+  #:use-module (guix utils)
   #:use-module (gnu packages autotools)
   #:use-module (gnu packages check)
+  #:use-module (gnu packages elf)
   #:use-module (gnu packages imagemagick)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages glib)
@@ -43,7 +46,8 @@ (define-module (gnu packages aidc)
   #:use-module (gnu packages qt)
   #:use-module (gnu packages video)
   #:use-module (guix build-system cmake)
-  #:use-module (guix build-system gnu))
+  #:use-module (guix build-system gnu)
+  #:use-module (guix build-system glib-or-gtk))
 
 (define-public zxing-cpp
   ;; Use the master branch as it includes unreleased build system improvements
@@ -171,6 +175,7 @@ (define-public libdmtx
 C/C++ programs to use its capabilities without restrictions or overhead.")
     (license license:bsd-3)))
 
+;; XXX: qt variant utils are broken: zbarcam-qt fails with segmentation fault.
 (define-public zbar
   (package
     (name "zbar")
@@ -184,11 +189,12 @@ (define-public zbar
          (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32
-         "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
-    (build-system gnu-build-system)
+        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
+    (build-system glib-or-gtk-build-system)
     (arguments
-     '(#:configure-flags (list "--with-gtk=auto"
+     (list
+      #:configure-flags '(list "--disable-static"
+                               "--with-gtk=auto"
                                "--with-python=auto"
                                (string-append "--with-dbusconfdir="
                                               (assoc-ref %outputs "out")
@@ -209,10 +215,10 @@ (define-public zbar
            libjpeg-turbo
            perl
            python
-           v4l-utils))
+           v4l-utils-minimal))
     (propagated-inputs
      ;; These are in 'requires' field of .pc files.
-     (list glib gtk+ qtbase-5))
+     (list glib gtk+))
     (synopsis "Bar code reader")
     (description
      "ZBar can read barcodes from various sources, such as video streams,
-- 
2.46.0

Message #101 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 04/10] gnu: zbar: Update to 0.23.92.
Date: Mon, 25 Nov 2024 10:52:37 +0100

* gnu/packages/aidc.scm (zbar): Update to 0.23.92.
---
 gnu/packages/aidc.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index dbbab0e22b..cf18801aec 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -179,7 +179,7 @@ (define-public libdmtx
 (define-public zbar
   (package
     (name "zbar")
-    (version "0.23.90")
+    (version "0.23.92")
     (source
      (origin
        (method git-fetch)
@@ -189,7 +189,7 @@ (define-public zbar
          (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
+        (base32 "0k3g0ql2m4dnflppp9r3k804d927g7zslczblzcrbvhp02g6n5an"))))
     (build-system glib-or-gtk-build-system)
     (arguments
      (list
-- 
2.46.0

Message #104 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 05/10] gnu: Add zbar-minimal.
Date: Mon, 25 Nov 2024 10:52:38 +0100

* gnu/packages/aidc.scm (zbar-minimal): New variable.
---
 gnu/packages/aidc.scm | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index cf18801aec..7add0f0a2f 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -231,6 +231,20 @@ (define-public zbar
     (home-page "https://github.com/mchehab/zbar")
     (license license:lgpl2.1+)))
 
+(define-public zbar-minimal
+  (package/inherit zbar
+    (name "zbar-minimal")
+    (build-system gnu-build-system)
+    (arguments
+     (substitute-keyword-arguments (package-arguments zbar)
+       ((#:configure-flags flags)
+        #~(cons* "--with-gtk=no" (delete "--with-gtk=auto" #$flags)))
+       ((#:disallowed-references _)
+        #~("qtbase" "gtk+"))))
+    (propagated-inputs
+     (modify-inputs (package-propagated-inputs zbar)
+       (delete "gtk+")))))
+
 (define-public qrcodegen-cpp
   (package
     (name "qrcodegen-cpp")
-- 
2.46.0

Message #107 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 06/10] gnu: libde265: Remove unused qtbase-5 input.
Date: Mon, 25 Nov 2024 10:52:39 +0100

* gnu/packages/video.scm (libde265)
[native-inputs]: Improve style.
[inputs]: Improve style.  Remove qtbase-5 which wasn't actually
used (need more inputs to build graphical tools).
---
 gnu/packages/video.scm | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 6f9eea6fc2..a2d7a1dfbb 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -711,16 +711,11 @@ (define-public libde265
      `(#:configure-flags
        (list "--disable-static")))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python" ,python-wrapper)))
+     (list autoconf automake libtool pkg-config python-wrapper))
     (inputs
-     `(;; XXX: Build fails with libvideogfx.
-       ;; ("libvideogfx" ,libvideogfx)
-       ("qt" ,qtbase-5)
-       ("sdl" ,sdl)))
+     ;; XXX: Build a complete version using libswscale or libvideogfx
+     ;; and qtbase-5.
+     (list sdl))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.46.0

Message #110 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 07/10] gnu: libde265: Update to 1.0.14.
Date: Mon, 25 Nov 2024 10:52:40 +0100

* gnu/packages/video.scm (libde265): Update to 1.0.14.
[inputs]: Replace sdl by sdl2.
---
 gnu/packages/video.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index a2d7a1dfbb..75fb77bb0d 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -695,7 +695,7 @@ (define-public libvideogfx
 (define-public libde265
   (package
     (name "libde265")
-    (version "1.0.8")
+    (version "1.0.14")
     (source
      (origin
        (method git-fetch)
@@ -705,7 +705,7 @@ (define-public libde265
          (commit (string-append "v" version))))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "1dzflqbk248lz5ws0ni5acmf32b3rmnq5gsfaz7691qqjxkl1zml"))))
+        (base32 "1a4n1hnr9ybji87irg2kya95slf4jnybnkg4x2zihnqqihbnv539"))))
     (build-system gnu-build-system)
     (arguments
      `(#:configure-flags
@@ -715,7 +715,7 @@ (define-public libde265
     (inputs
      ;; XXX: Build a complete version using libswscale or libvideogfx
      ;; and qtbase-5.
-     (list sdl))
+     (list sdl2))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.46.0

Message #113 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 08/10] gnu: libdc1394: Replace v4l-utils by
 v4l-utils-minimal.
Date: Mon, 25 Nov 2024 10:52:41 +0100

* gnu/packages/gstreamer.scm (libdc1394)[inputs]: Replace v4l-utils by
v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 13c8734e68..5ce33faa51 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -160,7 +160,7 @@ (define-public libdc1394
            libxv
            mesa
            sdl
-           v4l-utils))
+           v4l-utils-minimal))
     (synopsis "1394-Based Digital Camera Control Library")
     (description "LibDC1394 is a library that provides functionality to control
 any camera that conforms to the 1394-Based Digital Camera Specification written
-- 
2.46.0

Message #116 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 09/10] gnu: gst-plugins-bad: Replace inputs with -minimal
 variants.
Date: Mon, 25 Nov 2024 10:52:42 +0100

* gnu/packages/gstreamer.scm (gst-plugins-bad)[inputs]: Replace
v4l-utils by v4l-utils-minimal. Replace zbar by zbar-minimal.
---
 gnu/packages/gstreamer.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 5ce33faa51..bb1bdcb4a1 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -936,7 +936,7 @@ (define-public gst-plugins-bad
             tinyalsa
             transcode
             usrsctp
-            v4l-utils
+            v4l-utils-minimal
             vo-aacenc
             vo-amrwbenc
             vulkan-headers
@@ -946,7 +946,7 @@ (define-public gst-plugins-bad
             webrtc-audio-processing
             wildmidi
             wpebackend-fdo
-            zbar
+            zbar-minimal
             zxing-cpp-1.2)))
     (home-page "https://gstreamer.freedesktop.org/")
     (synopsis "Plugins for the GStreamer multimedia library")
-- 
2.46.0

Message #119 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, maxim.cournoyer <at> gmail.com
Subject: [PATCH v3 10/10] gnu: gst-plugins-good: Replace v4l-utils by
 v4l-utils-minimal.
Date: Mon, 25 Nov 2024 10:52:43 +0100

* gnu/packages/gstreamer.scm (gst-plugins-good)[inputs]: Replace
v4l-utils by v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index bb1bdcb4a1..905ac2d79a 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -732,7 +732,7 @@ (define libsoup
            speex
            taglib
            twolame
-           v4l-utils
+           v4l-utils-minimal
            wavpack
            zlib))
     (propagated-inputs
-- 
2.46.0

Message #122 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org, maxim.cournoyer <at> gmail.com
Subject: Re: [bug#74517] [PATCH v3 01/10] gnu: Add v4l-utils-minimal.
Date: Thu, 26 Dec 2024 23:12:23 +0100

Maxim, could you take a look at v3 of this patch series?

Thanks,
Ludo’.

Message #125 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH v3 01/10] gnu: Add v4l-utils-minimal.
Date: Sat, 28 Dec 2024 11:02:18 +0900

Hi Nicolas,

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> * gnu/packages/video.scm (v4l-utils-minimal): New variable.
> ---
>  gnu/packages/video.scm | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index 2da458ed01..6f9eea6fc2 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -4035,6 +4035,20 @@ (define-public v4l-utils
>      ;; libv4l2 is LGPL2.1+, while utilities are GPL2 only.
>      (license (list license:lgpl2.1+ license:gpl2))))
>  
> +(define-public v4l-utils-minimal
> +  (package/inherit v4l-utils
> +    (name "v4l-utils-minimal")
> +    (arguments
> +     (substitute-keyword-arguments (package-arguments v4l-utils)
> +       ((#:phases phases)
> +        #~(modify-phases #$phases
> +            (delete 'split)))
> +       ((#:disallowed-references _)
> +        #~("qtbase"))))

I think the value of #:disallowed-references need to be a list of
packages, not strings.

-- 
Thanks,
Maxim

Message #128 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH v3 05/10] gnu: Add zbar-minimal.
Date: Sat, 28 Dec 2024 11:03:36 +0900

Hi,

Nicolas Graves <ngraves <at> ngraves.fr> writes:

> * gnu/packages/aidc.scm (zbar-minimal): New variable.
> ---
>  gnu/packages/aidc.scm | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
>
> diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
> index cf18801aec..7add0f0a2f 100644
> --- a/gnu/packages/aidc.scm
> +++ b/gnu/packages/aidc.scm
> @@ -231,6 +231,20 @@ (define-public zbar
>      (home-page "https://github.com/mchehab/zbar")
>      (license license:lgpl2.1+)))
>  
> +(define-public zbar-minimal
> +  (package/inherit zbar
> +    (name "zbar-minimal")
> +    (build-system gnu-build-system)
> +    (arguments
> +     (substitute-keyword-arguments (package-arguments zbar)
> +       ((#:configure-flags flags)
> +        #~(cons* "--with-gtk=no" (delete "--with-gtk=auto" #$flags)))
> +       ((#:disallowed-references _)
> +        #~("qtbase" "gtk+"))))

Same comment here; it should be a list of packages; not a list of
strings, unless I've missed something.

-- 
Thanks,
Maxim

Message #131 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 74517 <at> debbugs.gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 Liliana Prikler <liliana.prikler <at> gmail.com>
Subject: Re: [bug#74517] [PATCH v3 01/10] gnu: Add v4l-utils-minimal.
Date: Sat, 28 Dec 2024 12:14:50 +0900

Hi Ludovic, Nicolas,

Ludovic Courtès <ludo <at> gnu.org> writes:

> Maxim, could you take a look at v3 of this patch series?

Thanks for the ping.  I've looked at v3, and it looks good to me, except
for the newly added #:disallowed-references (which I suggested adding)
which should be a list of packages, not a list of strings, if I'm not
mistaken.

Also, a #:disallowed-references added directly to GTK (to ensure it
continues not pulling in qtbase) would be a nice to have, but can be
added later too; I believe adding a disallowed reference should not lead
to rebuilds?

Nicolas, would you mind sending one last v4 with the corrected
#:disallowed-references?  You can grep for examples.  After which, I
think it should be merged to the gnome-team branch (CC'ing Liliana to
make sure it makes sense at this stage).

-- 
Thanks,
Maxim

Message #134 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 74517 <at> debbugs.gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 Liliana Prikler <liliana.prikler <at> gmail.com>
Subject: Re: [bug#74517] [PATCH v3 01/10] gnu: Add v4l-utils-minimal.
Date: Mon, 06 Jan 2025 16:07:42 +0100

Hi,

Maxim Cournoyer <maxim.cournoyer <at> gmail.com> skribis:

> Ludovic Courtès <ludo <at> gnu.org> writes:
>
>> Maxim, could you take a look at v3 of this patch series?
>
> Thanks for the ping.  I've looked at v3, and it looks good to me, except
> for the newly added #:disallowed-references (which I suggested adding)
> which should be a list of packages, not a list of strings, if I'm not
> mistaken.
>
> Also, a #:disallowed-references added directly to GTK (to ensure it
> continues not pulling in qtbase) would be a nice to have, but can be
> added later too; I believe adding a disallowed reference should not lead
> to rebuilds?

Add #:disallowed-references or #:allowed-references triggers changes the
derivation and thus triggers a rebuild.

> Nicolas, would you mind sending one last v4 with the corrected
> #:disallowed-references?  You can grep for examples.  After which, I
> think it should be merged to the gnome-team branch (CC'ing Liliana to
> make sure it makes sense at this stage).

Thanks,
Ludo’.

Message #137 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Ludovic Courtès <ludo <at> gnu.org>, Maxim Cournoyer
 <maxim.cournoyer <at> gmail.com>
Cc: 74517 <at> debbugs.gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>
Subject: Re: [bug#74517] [PATCH v3 01/10] gnu: Add v4l-utils-minimal.
Date: Mon, 06 Jan 2025 16:20:57 +0100

Hi,

Am Montag, dem 06.01.2025 um 16:07 +0100 schrieb Ludovic Courtès:
> Add[ing] #:disallowed-references or #:allowed-references triggers
> changes the derivation and thus triggers a rebuild.
> 
> > Nicolas, would you mind sending one last v4 with the corrected
> > #:disallowed-references?  You can grep for examples.  After which,
> > I think it should be merged to the gnome-team branch (CC'ing
> > Liliana to make sure it makes sense at this stage).
Yeah, we can rebuild GTK on gnome-team.  I previously submitted a
series to bump Webkit and we should probably bump all the other
packages to their latest minor version too.

Feel free to CC me for any patches that ought to go towards gnome-team
:)

Cheers

Message #140 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 02/10] gnu: zbar: Improve style.
Date: Tue,  7 Jan 2025 23:33:08 +0100

* gnu/packages/aidc.scm (zbar)[native-inputs, inputs]: Improve style.
---
 gnu/packages/aidc.scm | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index bc9c47a480..0c1f26495b 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -194,22 +194,22 @@ (define-public zbar
                                               (assoc-ref %outputs "out")
                                               "/etc"))))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("gettext" ,gettext-minimal)
-       ("glib" ,glib "bin")
-       ("gobject-introspection" ,gobject-introspection)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python-wrapper" ,python-wrapper)))
+     (list autoconf
+           automake
+           gettext-minimal
+           `(,glib "bin")
+           gobject-introspection
+           libtool
+           patchelf
+           pkg-config
+           python-wrapper))
     (inputs
-     `(("dbus" ,dbus)
-       ("imagemagick" ,imagemagick)
-       ("libjpeg" ,libjpeg-turbo)
-       ("perl" ,perl)
-       ("python" ,python)
-       ("qtx11extras" ,qtx11extras)
-       ("v4l-utils" ,v4l-utils)))
+     (list dbus
+           imagemagick
+           libjpeg-turbo
+           perl
+           python
+           v4l-utils))
     (propagated-inputs
      ;; These are in 'requires' field of .pc files.
      (list glib gtk+ qtbase-5))
-- 
2.47.1

Message #143 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 01/10] gnu: Add v4l-utils-minimal.
Date: Tue,  7 Jan 2025 23:33:07 +0100

* gnu/packages/video.scm (v4l-utils-minimal): New variable.
---
 gnu/packages/video.scm | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index ad0c6d63f0..37d0bbb52b 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -4052,6 +4052,20 @@ (define-public v4l-utils
     ;; libv4l2 is LGPL2.1+, while utilities are GPL2 only.
     (license (list license:lgpl2.1+ license:gpl2))))
 
+(define-public v4l-utils-minimal
+  (package/inherit v4l-utils
+    (name "v4l-utils-minimal")
+    (arguments
+     (substitute-keyword-arguments (package-arguments v4l-utils)
+       ((#:phases phases)
+        #~(modify-phases #$phases
+            (delete 'split)))
+       ((#:disallowed-references _)
+        (list qtbase))))
+    (outputs '("out"))
+    (inputs (modify-inputs (package-inputs v4l-utils)
+              (delete "qtbase")))))
+
 (define-public obs
   (package
     (name "obs")
-- 
2.47.1

Message #146 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 04/10] gnu: zbar: Update to 0.23.92.
Date: Tue,  7 Jan 2025 23:33:10 +0100

* gnu/packages/aidc.scm (zbar): Update to 0.23.92.
---
 gnu/packages/aidc.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index dbbab0e22b..cf18801aec 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -179,7 +179,7 @@ (define-public libdmtx
 (define-public zbar
   (package
     (name "zbar")
-    (version "0.23.90")
+    (version "0.23.92")
     (source
      (origin
        (method git-fetch)
@@ -189,7 +189,7 @@ (define-public zbar
          (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
+        (base32 "0k3g0ql2m4dnflppp9r3k804d927g7zslczblzcrbvhp02g6n5an"))))
     (build-system glib-or-gtk-build-system)
     (arguments
      (list
-- 
2.47.1

Message #149 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 03/10] gnu: zbar: Disable static and qt build.
Date: Tue,  7 Jan 2025 23:33:09 +0100

* gnu/packages/aidc.scm (zbar): Disable static and qt build.
[build-system]: Switch to glib-or-gtk-build-system.
[arguments]<#:configure-flags>: Add --disable-static.
[inputs]: Replace v4l-utils by v4l-utils-minimal.
---
 gnu/packages/aidc.scm | 20 +++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index 0c1f26495b..dbbab0e22b 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -5,6 +5,7 @@
 ;;; Copyright © 2018, 2019, 2022 Tobias Geerinckx-Rice <me <at> tobias.gr>
 ;;; Copyright © 2019 Guillaume Le Vaillant <glv <at> posteo.net>
 ;;; Copyright © 2020 Leo Famulari <leo <at> famulari.name>
+;;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -28,8 +29,10 @@ (define-module (gnu packages aidc)
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix git-download)
+  #:use-module (guix utils)
   #:use-module (gnu packages autotools)
   #:use-module (gnu packages check)
+  #:use-module (gnu packages elf)
   #:use-module (gnu packages imagemagick)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages glib)
@@ -43,7 +46,8 @@ (define-module (gnu packages aidc)
   #:use-module (gnu packages qt)
   #:use-module (gnu packages video)
   #:use-module (guix build-system cmake)
-  #:use-module (guix build-system gnu))
+  #:use-module (guix build-system gnu)
+  #:use-module (guix build-system glib-or-gtk))
 
 (define-public zxing-cpp
   ;; Use the master branch as it includes unreleased build system improvements
@@ -171,6 +175,7 @@ (define-public libdmtx
 C/C++ programs to use its capabilities without restrictions or overhead.")
     (license license:bsd-3)))
 
+;; XXX: qt variant utils are broken: zbarcam-qt fails with segmentation fault.
 (define-public zbar
   (package
     (name "zbar")
@@ -184,11 +189,12 @@ (define-public zbar
          (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32
-         "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
-    (build-system gnu-build-system)
+        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
+    (build-system glib-or-gtk-build-system)
     (arguments
-     '(#:configure-flags (list "--with-gtk=auto"
+     (list
+      #:configure-flags '(list "--disable-static"
+                               "--with-gtk=auto"
                                "--with-python=auto"
                                (string-append "--with-dbusconfdir="
                                               (assoc-ref %outputs "out")
@@ -209,10 +215,10 @@ (define-public zbar
            libjpeg-turbo
            perl
            python
-           v4l-utils))
+           v4l-utils-minimal))
     (propagated-inputs
      ;; These are in 'requires' field of .pc files.
-     (list glib gtk+ qtbase-5))
+     (list glib gtk+))
     (synopsis "Bar code reader")
     (description
      "ZBar can read barcodes from various sources, such as video streams,
-- 
2.47.1

Message #152 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 05/10] gnu: Add zbar-minimal.
Date: Tue,  7 Jan 2025 23:33:11 +0100

* gnu/packages/aidc.scm (zbar-minimal): New variable.
---
 gnu/packages/aidc.scm | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index cf18801aec..e4ebf72f41 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -231,6 +231,20 @@ (define-public zbar
     (home-page "https://github.com/mchehab/zbar")
     (license license:lgpl2.1+)))
 
+(define-public zbar-minimal
+  (package/inherit zbar
+    (name "zbar-minimal")
+    (build-system gnu-build-system)
+    (arguments
+     (substitute-keyword-arguments (package-arguments zbar)
+       ((#:configure-flags flags)
+        #~(cons* "--with-gtk=no" (delete "--with-gtk=auto" #$flags)))
+       ((#:disallowed-references _)
+        (list qtbase gtk+))))
+    (propagated-inputs
+     (modify-inputs (package-propagated-inputs zbar)
+       (delete "gtk+")))))
+
 (define-public qrcodegen-cpp
   (package
     (name "qrcodegen-cpp")
-- 
2.47.1

Message #155 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 06/10] gnu: libde265: Remove unused qtbase-5 input.
Date: Tue,  7 Jan 2025 23:33:12 +0100

* gnu/packages/video.scm (libde265)
[native-inputs]: Improve style.
[inputs]: Improve style.  Remove qtbase-5 which wasn't actually
used (need more inputs to build graphical tools).
---
 gnu/packages/video.scm | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 37d0bbb52b..72c89cc851 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -711,16 +711,11 @@ (define-public libde265
      `(#:configure-flags
        (list "--disable-static")))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python" ,python-wrapper)))
+     (list autoconf automake libtool pkg-config python-wrapper))
     (inputs
-     `(;; XXX: Build fails with libvideogfx.
-       ;; ("libvideogfx" ,libvideogfx)
-       ("qt" ,qtbase-5)
-       ("sdl" ,sdl)))
+     ;; XXX: Build a complete version using libswscale or libvideogfx
+     ;; and qtbase-5.
+     (list sdl))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.47.1

Message #158 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 07/10] gnu: libde265: Update to 1.0.14.
Date: Tue,  7 Jan 2025 23:33:13 +0100

* gnu/packages/video.scm (libde265): Update to 1.0.14.
[inputs]: Replace sdl by sdl2.
---
 gnu/packages/video.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 72c89cc851..c9284160fb 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -695,7 +695,7 @@ (define-public libvideogfx
 (define-public libde265
   (package
     (name "libde265")
-    (version "1.0.8")
+    (version "1.0.14")
     (source
      (origin
        (method git-fetch)
@@ -705,7 +705,7 @@ (define-public libde265
          (commit (string-append "v" version))))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "1dzflqbk248lz5ws0ni5acmf32b3rmnq5gsfaz7691qqjxkl1zml"))))
+        (base32 "1a4n1hnr9ybji87irg2kya95slf4jnybnkg4x2zihnqqihbnv539"))))
     (build-system gnu-build-system)
     (arguments
      `(#:configure-flags
@@ -715,7 +715,7 @@ (define-public libde265
     (inputs
      ;; XXX: Build a complete version using libswscale or libvideogfx
      ;; and qtbase-5.
-     (list sdl))
+     (list sdl2))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.47.1

Message #161 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 08/10] gnu: libdc1394: Replace v4l-utils by
 v4l-utils-minimal.
Date: Tue,  7 Jan 2025 23:33:14 +0100

* gnu/packages/gstreamer.scm (libdc1394)[inputs]: Replace v4l-utils by
v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index c4fb8b9b4d..6d5b8b894c 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -160,7 +160,7 @@ (define-public libdc1394
            libxv
            mesa
            sdl
-           v4l-utils))
+           v4l-utils-minimal))
     (synopsis "1394-Based Digital Camera Control Library")
     (description "LibDC1394 is a library that provides functionality to control
 any camera that conforms to the 1394-Based Digital Camera Specification written
-- 
2.47.1

Message #164 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 09/10] gnu: gst-plugins-bad: Replace inputs with -minimal
 variants.
Date: Tue,  7 Jan 2025 23:33:15 +0100

* gnu/packages/gstreamer.scm (gst-plugins-bad)[inputs]: Replace
v4l-utils by v4l-utils-minimal. Replace zbar by zbar-minimal.
---
 gnu/packages/gstreamer.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 6d5b8b894c..084c9f70c5 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -936,7 +936,7 @@ (define-public gst-plugins-bad
             tinyalsa
             transcode
             usrsctp
-            v4l-utils
+            v4l-utils-minimal
             vo-aacenc
             vo-amrwbenc
             vulkan-headers
@@ -946,7 +946,7 @@ (define-public gst-plugins-bad
             webrtc-audio-processing
             wildmidi
             wpebackend-fdo
-            zbar
+            zbar-minimal
             zxing-cpp-1.2)))
     (home-page "https://gstreamer.freedesktop.org/")
     (synopsis "Plugins for the GStreamer multimedia library")
-- 
2.47.1

Message #167 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v4 10/10] gnu: gst-plugins-good: Replace v4l-utils by
 v4l-utils-minimal.
Date: Tue,  7 Jan 2025 23:33:16 +0100

* gnu/packages/gstreamer.scm (gst-plugins-good)[inputs]: Replace
v4l-utils by v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 084c9f70c5..837e360496 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -732,7 +732,7 @@ (define libsoup
            speex
            taglib
            twolame
-           v4l-utils
+           v4l-utils-minimal
            wavpack
            zlib))
     (propagated-inputs
-- 
2.47.1

Message #170 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>, 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, maxim.cournoyer <at> gmail.com
Subject: Re: [PATCH v4 05/10] gnu: Add zbar-minimal.
Date: Wed, 08 Jan 2025 06:13:05 +0100

Am Dienstag, dem 07.01.2025 um 23:33 +0100 schrieb Nicolas Graves:
> * gnu/packages/aidc.scm (zbar-minimal): New variable.
> ---
>  gnu/packages/aidc.scm | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
> index cf18801aec..e4ebf72f41 100644
> --- a/gnu/packages/aidc.scm
> +++ b/gnu/packages/aidc.scm
> @@ -231,6 +231,20 @@ (define-public zbar
>      (home-page "https://github.com/mchehab/zbar")
>      (license license:lgpl2.1+)))
>  
> +(define-public zbar-minimal
> +  (package/inherit zbar
> +    (name "zbar-minimal")
> +    (build-system gnu-build-system)
> +    (arguments
> +     (substitute-keyword-arguments (package-arguments zbar)
> +       ((#:configure-flags flags)
> +        #~(cons* "--with-gtk=no" (delete "--with-gtk=auto"
> #$flags)))
> +       ((#:disallowed-references _)
> +        (list qtbase gtk+))))
I think this might not have an effect if disallowed-references aren't
provided in zbar (which IIUC they're not).  You can add '() as a third
parameter to always have this code live.
> +    (propagated-inputs
> +     (modify-inputs (package-propagated-inputs zbar)
> +       (delete "gtk+")))))
> +
>  (define-public qrcodegen-cpp
>    (package
>      (name "qrcodegen-cpp")
Cheers

Message #173 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 01/10] gnu: Add v4l-utils-minimal.
Date: Wed,  8 Jan 2025 07:11:34 +0100

* gnu/packages/video.scm (v4l-utils-minimal): New variable.
---
 gnu/packages/video.scm | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 5c878bfcb9..b91865d6af 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -4052,6 +4052,20 @@ (define-public v4l-utils
     ;; libv4l2 is LGPL2.1+, while utilities are GPL2 only.
     (license (list license:lgpl2.1+ license:gpl2))))
 
+(define-public v4l-utils-minimal
+  (package/inherit v4l-utils
+    (name "v4l-utils-minimal")
+    (arguments
+     (substitute-keyword-arguments (package-arguments v4l-utils)
+       ((#:phases phases)
+        #~(modify-phases #$phases
+            (delete 'split)))
+       ((#:disallowed-references _ '())
+        (list qtbase qtbase-5))))
+    (outputs '("out"))
+    (inputs (modify-inputs (package-inputs v4l-utils)
+              (delete "qtbase")))))
+
 (define-public obs
   (package
     (name "obs")
-- 
2.47.1

Message #176 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 02/10] gnu: zbar: Improve style.
Date: Wed,  8 Jan 2025 07:11:35 +0100

* gnu/packages/aidc.scm (zbar)[native-inputs, inputs]: Improve style.
---
 gnu/packages/aidc.scm | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index bc9c47a480..0c1f26495b 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -194,22 +194,22 @@ (define-public zbar
                                               (assoc-ref %outputs "out")
                                               "/etc"))))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("gettext" ,gettext-minimal)
-       ("glib" ,glib "bin")
-       ("gobject-introspection" ,gobject-introspection)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python-wrapper" ,python-wrapper)))
+     (list autoconf
+           automake
+           gettext-minimal
+           `(,glib "bin")
+           gobject-introspection
+           libtool
+           patchelf
+           pkg-config
+           python-wrapper))
     (inputs
-     `(("dbus" ,dbus)
-       ("imagemagick" ,imagemagick)
-       ("libjpeg" ,libjpeg-turbo)
-       ("perl" ,perl)
-       ("python" ,python)
-       ("qtx11extras" ,qtx11extras)
-       ("v4l-utils" ,v4l-utils)))
+     (list dbus
+           imagemagick
+           libjpeg-turbo
+           perl
+           python
+           v4l-utils))
     (propagated-inputs
      ;; These are in 'requires' field of .pc files.
      (list glib gtk+ qtbase-5))
-- 
2.47.1

Message #179 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 03/10] gnu: zbar: Disable static and qt build.
Date: Wed,  8 Jan 2025 07:11:36 +0100

* gnu/packages/aidc.scm (zbar): Disable static and qt build.
[build-system]: Switch to glib-or-gtk-build-system.
[arguments]<#:configure-flags>: Add --disable-static.
[inputs]: Replace v4l-utils by v4l-utils-minimal.
---
 gnu/packages/aidc.scm | 20 +++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index 0c1f26495b..dbbab0e22b 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -5,6 +5,7 @@
 ;;; Copyright © 2018, 2019, 2022 Tobias Geerinckx-Rice <me <at> tobias.gr>
 ;;; Copyright © 2019 Guillaume Le Vaillant <glv <at> posteo.net>
 ;;; Copyright © 2020 Leo Famulari <leo <at> famulari.name>
+;;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -28,8 +29,10 @@ (define-module (gnu packages aidc)
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix git-download)
+  #:use-module (guix utils)
   #:use-module (gnu packages autotools)
   #:use-module (gnu packages check)
+  #:use-module (gnu packages elf)
   #:use-module (gnu packages imagemagick)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages glib)
@@ -43,7 +46,8 @@ (define-module (gnu packages aidc)
   #:use-module (gnu packages qt)
   #:use-module (gnu packages video)
   #:use-module (guix build-system cmake)
-  #:use-module (guix build-system gnu))
+  #:use-module (guix build-system gnu)
+  #:use-module (guix build-system glib-or-gtk))
 
 (define-public zxing-cpp
   ;; Use the master branch as it includes unreleased build system improvements
@@ -171,6 +175,7 @@ (define-public libdmtx
 C/C++ programs to use its capabilities without restrictions or overhead.")
     (license license:bsd-3)))
 
+;; XXX: qt variant utils are broken: zbarcam-qt fails with segmentation fault.
 (define-public zbar
   (package
     (name "zbar")
@@ -184,11 +189,12 @@ (define-public zbar
          (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32
-         "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
-    (build-system gnu-build-system)
+        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
+    (build-system glib-or-gtk-build-system)
     (arguments
-     '(#:configure-flags (list "--with-gtk=auto"
+     (list
+      #:configure-flags '(list "--disable-static"
+                               "--with-gtk=auto"
                                "--with-python=auto"
                                (string-append "--with-dbusconfdir="
                                               (assoc-ref %outputs "out")
@@ -209,10 +215,10 @@ (define-public zbar
            libjpeg-turbo
            perl
            python
-           v4l-utils))
+           v4l-utils-minimal))
     (propagated-inputs
      ;; These are in 'requires' field of .pc files.
-     (list glib gtk+ qtbase-5))
+     (list glib gtk+))
     (synopsis "Bar code reader")
     (description
      "ZBar can read barcodes from various sources, such as video streams,
-- 
2.47.1

Message #182 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 04/10] gnu: zbar: Update to 0.23.92.
Date: Wed,  8 Jan 2025 07:11:37 +0100

* gnu/packages/aidc.scm (zbar): Update to 0.23.92.
---
 gnu/packages/aidc.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index dbbab0e22b..cf18801aec 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -179,7 +179,7 @@ (define-public libdmtx
 (define-public zbar
   (package
     (name "zbar")
-    (version "0.23.90")
+    (version "0.23.92")
     (source
      (origin
        (method git-fetch)
@@ -189,7 +189,7 @@ (define-public zbar
          (commit version)))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "0rf3i7lx0fqzxsngird6l4d4dnl612nr32rm8sib699qqx67px8n"))))
+        (base32 "0k3g0ql2m4dnflppp9r3k804d927g7zslczblzcrbvhp02g6n5an"))))
     (build-system glib-or-gtk-build-system)
     (arguments
      (list
-- 
2.47.1

Message #185 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 05/10] gnu: Add zbar-minimal.
Date: Wed,  8 Jan 2025 07:11:38 +0100

* gnu/packages/aidc.scm (zbar-minimal): New variable.
---
 gnu/packages/aidc.scm | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/gnu/packages/aidc.scm b/gnu/packages/aidc.scm
index cf18801aec..2ba6e88412 100644
--- a/gnu/packages/aidc.scm
+++ b/gnu/packages/aidc.scm
@@ -231,6 +231,20 @@ (define-public zbar
     (home-page "https://github.com/mchehab/zbar")
     (license license:lgpl2.1+)))
 
+(define-public zbar-minimal
+  (package/inherit zbar
+    (name "zbar-minimal")
+    (build-system gnu-build-system)
+    (arguments
+     (substitute-keyword-arguments (package-arguments zbar)
+       ((#:configure-flags flags)
+        #~(cons* "--with-gtk=no" (delete "--with-gtk=auto" #$flags)))
+       ((#:disallowed-references _ '())
+        (list qtbase gtk+))))
+    (propagated-inputs
+     (modify-inputs (package-propagated-inputs zbar)
+       (delete "gtk+")))))
+
 (define-public qrcodegen-cpp
   (package
     (name "qrcodegen-cpp")
-- 
2.47.1

Message #188 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 06/10] gnu: libde265: Remove unused qtbase-5 input.
Date: Wed,  8 Jan 2025 07:11:39 +0100

* gnu/packages/video.scm (libde265)
[native-inputs]: Improve style.
[inputs]: Improve style.  Remove qtbase-5 which wasn't actually
used (need more inputs to build graphical tools).
---
 gnu/packages/video.scm | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index b91865d6af..9375c5ba61 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -711,16 +711,11 @@ (define-public libde265
      `(#:configure-flags
        (list "--disable-static")))
     (native-inputs
-     `(("autoconf" ,autoconf)
-       ("automake" ,automake)
-       ("libtool" ,libtool)
-       ("pkg-config" ,pkg-config)
-       ("python" ,python-wrapper)))
+     (list autoconf automake libtool pkg-config python-wrapper))
     (inputs
-     `(;; XXX: Build fails with libvideogfx.
-       ;; ("libvideogfx" ,libvideogfx)
-       ("qt" ,qtbase-5)
-       ("sdl" ,sdl)))
+     ;; XXX: Build a complete version using libswscale or libvideogfx
+     ;; and qtbase-5.
+     (list sdl))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.47.1

Message #191 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 07/10] gnu: libde265: Update to 1.0.14.
Date: Wed,  8 Jan 2025 07:11:40 +0100

* gnu/packages/video.scm (libde265): Update to 1.0.14.
[inputs]: Replace sdl by sdl2.
---
 gnu/packages/video.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 9375c5ba61..13a6429895 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -695,7 +695,7 @@ (define-public libvideogfx
 (define-public libde265
   (package
     (name "libde265")
-    (version "1.0.8")
+    (version "1.0.14")
     (source
      (origin
        (method git-fetch)
@@ -705,7 +705,7 @@ (define-public libde265
          (commit (string-append "v" version))))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "1dzflqbk248lz5ws0ni5acmf32b3rmnq5gsfaz7691qqjxkl1zml"))))
+        (base32 "1a4n1hnr9ybji87irg2kya95slf4jnybnkg4x2zihnqqihbnv539"))))
     (build-system gnu-build-system)
     (arguments
      `(#:configure-flags
@@ -715,7 +715,7 @@ (define-public libde265
     (inputs
      ;; XXX: Build a complete version using libswscale or libvideogfx
      ;; and qtbase-5.
-     (list sdl))
+     (list sdl2))
     (synopsis "H.265 video codec implementation")
     (description "Libde265 is an implementation of the h.265 video codec.  It is
 written from scratch and has a plain C API to enable a simple integration into
-- 
2.47.1

Message #194 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 08/10] gnu: libdc1394: Replace v4l-utils by
 v4l-utils-minimal.
Date: Wed,  8 Jan 2025 07:11:41 +0100

* gnu/packages/gstreamer.scm (libdc1394)[inputs]: Replace v4l-utils by
v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index c4fb8b9b4d..6d5b8b894c 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -160,7 +160,7 @@ (define-public libdc1394
            libxv
            mesa
            sdl
-           v4l-utils))
+           v4l-utils-minimal))
     (synopsis "1394-Based Digital Camera Control Library")
     (description "LibDC1394 is a library that provides functionality to control
 any camera that conforms to the 1394-Based Digital Camera Specification written
-- 
2.47.1

Message #197 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 09/10] gnu: gst-plugins-bad: Replace inputs with -minimal
 variants.
Date: Wed,  8 Jan 2025 07:11:42 +0100

* gnu/packages/gstreamer.scm (gst-plugins-bad)[inputs]: Replace
v4l-utils by v4l-utils-minimal. Replace zbar by zbar-minimal.
---
 gnu/packages/gstreamer.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 6d5b8b894c..084c9f70c5 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -936,7 +936,7 @@ (define-public gst-plugins-bad
             tinyalsa
             transcode
             usrsctp
-            v4l-utils
+            v4l-utils-minimal
             vo-aacenc
             vo-amrwbenc
             vulkan-headers
@@ -946,7 +946,7 @@ (define-public gst-plugins-bad
             webrtc-audio-processing
             wildmidi
             wpebackend-fdo
-            zbar
+            zbar-minimal
             zxing-cpp-1.2)))
     (home-page "https://gstreamer.freedesktop.org/")
     (synopsis "Plugins for the GStreamer multimedia library")
-- 
2.47.1

Message #200 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74517 <at> debbugs.gnu.org
Cc: ludo <at> gnu.org, Nicolas Graves <ngraves <at> ngraves.fr>,
 liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: [PATCH v5 10/10] gnu: gst-plugins-good: Replace v4l-utils by
 v4l-utils-minimal.
Date: Wed,  8 Jan 2025 07:11:43 +0100

* gnu/packages/gstreamer.scm (gst-plugins-good)[inputs]: Replace
v4l-utils by v4l-utils-minimal.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 084c9f70c5..837e360496 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -732,7 +732,7 @@ (define libsoup
            speex
            taglib
            twolame
-           v4l-utils
+           v4l-utils-minimal
            wavpack
            zlib))
     (propagated-inputs
-- 
2.47.1

Message #203 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org, liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com
Subject: Re: [PATCH v5 01/10] gnu: Add v4l-utils-minimal.
Date: Wed, 08 Jan 2025 11:26:50 +0100

Hello,

Nicolas Graves <ngraves <at> ngraves.fr> skribis:

> * gnu/packages/video.scm (v4l-utils-minimal): New variable.

Side note: I wonder why I’m receiving these patches since this is not in
the scope of the teams I’m in, I think?

Ludo’.

Message #206 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org
Subject: Re: [bug#74517] [PATCH v5 01/10] gnu: Add v4l-utils-minimal.
Date: Thu, 09 Jan 2025 14:29:02 +0100

Nicolas Graves <ngraves <at> ngraves.fr> skribis:

> On 2025-01-08 11:26, Ludovic Courtès wrote:
>
>> Hello,
>>
>> Nicolas Graves <ngraves <at> ngraves.fr> skribis:
>>
>>> * gnu/packages/video.scm (v4l-utils-minimal): New variable.
>>
>> Side note: I wonder why I’m receiving these patches since this is not in
>> the scope of the teams I’m in, I think?
>
> Sorry, I think I put in cc people who participated by hand. 

No worries.

I was surprised because normally Git hooks are set up automatically such
that the relevant teams are Cc’d.  (Perhaps there are no relevant teams
in this case?)

Ludo’.

Message #209 received at 74517 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74517 <at> debbugs.gnu.org, ludo <at> gnu.org, liliana.prikler <at> gmail.com
Subject: Re: [PATCH v5 01/10] gnu: Add v4l-utils-minimal.
Date: Fri, 17 Jan 2025 13:14:59 +0900

Hi,

I think this LGTM, in its version v5.  Liliana, can we apply it to the
gnome-team branch to avoid disrupting master with many rebuilds?

-- 
Thanks,
Maxim

Message #214 received at 74517-done <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>, Nicolas Graves
 <ngraves <at> ngraves.fr>
Cc: ludo <at> gnu.org, 74517-done <at> debbugs.gnu.org
Subject: Re: [PATCH v5 01/10] gnu: Add v4l-utils-minimal.
Date: Sat, 18 Jan 2025 11:45:19 +0100

Am Freitag, dem 17.01.2025 um 13:14 +0900 schrieb Maxim Cournoyer:
> Hi,
> 
> I think this LGTM, in its version v5.  Liliana, can we apply it to
> the gnome-team branch to avoid disrupting master with many rebuilds?
Sure, consider it done.

Cheers

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.