GNU bug report logs - #72338
[PATCH] gnu: readymedia: Add coreutils to dependencies.

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 72338 in the body.
You can then email your comments to 72338 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: Fabio Natali <me <at> fabionatali.com>
To: guix-patches <at> gnu.org
Cc: Fabio Natali <me <at> fabionatali.com>
Subject: [PATCH] gnu: readymedia: Add coreutils to dependencies.
Date: Sun, 28 Jul 2024 16:20:30 +0100

* gnu/packages/upnp.scm (readymedia)[inputs]: Add coreutils.

Change-Id: I8f73802bd0d8fecbf5ee20853eccefc2eef82411
---
Hi,

This is to add coreutils to ReadyMedia/MiniDLNA's dependencies. Without
coreutils (and the 'rm' command in particular) some ReadyMedia operations would
otherwise fail.

Thanks, cheers, Fabio.

 gnu/packages/upnp.scm | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/upnp.scm b/gnu/packages/upnp.scm
index aab68a6414..17605d3b96 100644
--- a/gnu/packages/upnp.scm
+++ b/gnu/packages/upnp.scm
@@ -26,6 +26,7 @@
 (define-module (gnu packages upnp)
   #:use-module (gnu packages)
   #:use-module (gnu packages autotools)
+  #:use-module (gnu packages base)
   #:use-module (gnu packages check)
   #:use-module (gnu packages compression)
   #:use-module (gnu packages curl)
@@ -154,12 +155,17 @@ (define-public readymedia
        #:phases
        (modify-phases %standard-phases
          (add-after 'unpack 'patch-source
-           (lambda _
+           (lambda* (#:key inputs #:allow-other-keys)
              (substitute* "Makefile.am"
-               ((".*LIBAVUTIL_LIBS.*") "")))))))
+               ((".*LIBAVUTIL_LIBS.*") ""))
+             (substitute* "minidlna.c"
+               (("rm -rf")
+                (string-append
+                 (assoc-ref inputs "coreutils") "/bin/rm -rf"))))))))
     (native-inputs (list autoconf automake gettext-minimal))
     (inputs
-     (list ffmpeg
+     (list coreutils
+           ffmpeg
            flac
            libexif
            libid3tag

base-commit: 46a64c7fdd057283063aae6df058579bb07c4b6a
-- 
2.45.2

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

From: Zheng Junjie <zhengjunjie <at> iscas.ac.cn>
To: Fabio Natali via Guix-patches via <guix-patches <at> gnu.org>
Cc: 72338 <at> debbugs.gnu.org, Fabio Natali <me <at> fabionatali.com>
Subject: Re: [bug#72338] [PATCH] gnu: readymedia: Add coreutils to
 dependencies.
Date: Tue, 30 Jul 2024 13:17:34 +0800

[Message part 1 (text/plain, inline)]

Fabio Natali via Guix-patches via <guix-patches <at> gnu.org> writes:

> * gnu/packages/upnp.scm (readymedia)[inputs]: Add coreutils.
>
> Change-Id: I8f73802bd0d8fecbf5ee20853eccefc2eef82411
> ---
> Hi,
>
> This is to add coreutils to ReadyMedia/MiniDLNA's dependencies. Without
> coreutils (and the 'rm' command in particular) some ReadyMedia operations would
> otherwise fail.
>
> Thanks, cheers, Fabio.
>
>  gnu/packages/upnp.scm | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/upnp.scm b/gnu/packages/upnp.scm
> index aab68a6414..17605d3b96 100644
> --- a/gnu/packages/upnp.scm
> +++ b/gnu/packages/upnp.scm
> @@ -26,6 +26,7 @@
>  (define-module (gnu packages upnp)
>    #:use-module (gnu packages)
>    #:use-module (gnu packages autotools)
> +  #:use-module (gnu packages base)
>    #:use-module (gnu packages check)
>    #:use-module (gnu packages compression)
>    #:use-module (gnu packages curl)
> @@ -154,12 +155,17 @@ (define-public readymedia
>         #:phases
>         (modify-phases %standard-phases
>           (add-after 'unpack 'patch-source
> -           (lambda _
> +           (lambda* (#:key inputs #:allow-other-keys)
>               (substitute* "Makefile.am"
> -               ((".*LIBAVUTIL_LIBS.*") "")))))))
> +               ((".*LIBAVUTIL_LIBS.*") ""))
> +             (substitute* "minidlna.c"
> +               (("rm -rf")
> +                (string-append
> +                 (assoc-ref inputs "coreutils") "/bin/rm -rf"))))))))

use (string-append (search-input-file inputs "/bin/rm") " -rf")

>      (native-inputs (list autoconf automake gettext-minimal))
>      (inputs
> -     (list ffmpeg
> +     (list coreutils

please use coreutils-minimal

> +           ffmpeg
>             flac
>             libexif
>             libid3tag
>
> base-commit: 46a64c7fdd057283063aae6df058579bb07c4b6a

[signature.asc (application/pgp-signature, inline)]

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

From: Fabio Natali <me <at> fabionatali.com>
To: 72338 <at> debbugs.gnu.org
Cc: zhengjunjie <at> iscas.ac.cn, Fabio Natali <me <at> fabionatali.com>
Subject: [PATCH] gnu: readymedia: Add coreutils to dependencies.
Date: Tue, 30 Jul 2024 11:58:03 +0100

* gnu/packages/upnp.scm (readymedia)[inputs]: Add coreutils.

Change-Id: I8f73802bd0d8fecbf5ee20853eccefc2eef82411
---
Hi Zheng,

Thanks for reviewing my patch. Please find below an updated version. Let me know
should you spot anything else. 🙏

Thanks, cheers, Fabio.

 gnu/packages/upnp.scm | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/upnp.scm b/gnu/packages/upnp.scm
index aab68a6414..d85e1da956 100644
--- a/gnu/packages/upnp.scm
+++ b/gnu/packages/upnp.scm
@@ -26,6 +26,7 @@
 (define-module (gnu packages upnp)
   #:use-module (gnu packages)
   #:use-module (gnu packages autotools)
+  #:use-module (gnu packages base)
   #:use-module (gnu packages check)
   #:use-module (gnu packages compression)
   #:use-module (gnu packages curl)
@@ -154,12 +155,17 @@ (define-public readymedia
        #:phases
        (modify-phases %standard-phases
          (add-after 'unpack 'patch-source
-           (lambda _
+           (lambda* (#:key inputs #:allow-other-keys)
              (substitute* "Makefile.am"
-               ((".*LIBAVUTIL_LIBS.*") "")))))))
+               ((".*LIBAVUTIL_LIBS.*") ""))
+             (substitute* "minidlna.c"
+               (("rm -rf")
+                (string-append
+                 (search-input-file inputs "/bin/rm") " -rf"))))))))
     (native-inputs (list autoconf automake gettext-minimal))
     (inputs
-     (list ffmpeg
+     (list coreutils-minimal
+           ffmpeg
            flac
            libexif
            libid3tag

base-commit: 46a64c7fdd057283063aae6df058579bb07c4b6a
-- 
2.45.2

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

From: Z572 <zhengjunjie <at> iscas.ac.cn>
To: Fabio Natali via Guix-patches via <guix-patches <at> gnu.org>
Cc: 72338-done <at> debbugs.gnu.org, Fabio Natali <me <at> fabionatali.com>
Subject: Re: [bug#72338] [PATCH] gnu: readymedia: Add coreutils to
 dependencies.
Date: Thu, 01 Aug 2024 23:55:18 +0800

[Message part 1 (text/plain, inline)]

Fabio Natali via Guix-patches via <guix-patches <at> gnu.org> writes:

> * gnu/packages/upnp.scm (readymedia)[inputs]: Add coreutils.
>
> Change-Id: I8f73802bd0d8fecbf5ee20853eccefc2eef82411
> ---
> Hi Zheng,
>
> Thanks for reviewing my patch. Please find below an updated version. Let me know
> should you spot anything else. 🙏
>
> Thanks, cheers, Fabio.
>
>  gnu/packages/upnp.scm | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/upnp.scm b/gnu/packages/upnp.scm
> index aab68a6414..d85e1da956 100644
> --- a/gnu/packages/upnp.scm
> +++ b/gnu/packages/upnp.scm
> @@ -26,6 +26,7 @@
>  (define-module (gnu packages upnp)
>    #:use-module (gnu packages)
>    #:use-module (gnu packages autotools)
> +  #:use-module (gnu packages base)
>    #:use-module (gnu packages check)
>    #:use-module (gnu packages compression)
>    #:use-module (gnu packages curl)
> @@ -154,12 +155,17 @@ (define-public readymedia
>         #:phases
>         (modify-phases %standard-phases
>           (add-after 'unpack 'patch-source
> -           (lambda _
> +           (lambda* (#:key inputs #:allow-other-keys)
>               (substitute* "Makefile.am"
> -               ((".*LIBAVUTIL_LIBS.*") "")))))))
> +               ((".*LIBAVUTIL_LIBS.*") ""))
> +             (substitute* "minidlna.c"
> +               (("rm -rf")
> +                (string-append
> +                 (search-input-file inputs "/bin/rm") " -rf"))))))))
>      (native-inputs (list autoconf automake gettext-minimal))
>      (inputs
> -     (list ffmpeg
> +     (list coreutils-minimal
> +           ffmpeg
>             flac
>             libexif
>             libid3tag
>
> base-commit: 46a64c7fdd057283063aae6df058579bb07c4b6a

Thanks, push, close.

[signature.asc (application/pgp-signature, inline)]

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

From: Fabio Natali <me <at> fabionatali.com>
To: Z572 <zhengjunjie <at> iscas.ac.cn>, Fabio Natali via Guix-patches via
 <guix-patches <at> gnu.org>
Cc: 72338-done <at> debbugs.gnu.org
Subject: Re: [bug#72338] [PATCH] gnu: readymedia: Add coreutils to
 dependencies.
Date: Thu, 01 Aug 2024 17:54:16 +0100

On 2024-08-01, 23:55 +0800, Z572 <zhengjunjie <at> iscas.ac.cn> wrote:
> Thanks, push, close.

This is brilliant! Thanks Zheng, much appreciated. Cheers, Fabio.

Previous Next

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