[bug#30677] [PATCH] gnu: youtube-viewer: Add '$out/lib/perl5/site_perl/' to search path.

[bug#30677] [PATCH] gnu: youtube-viewer: Add '$out/lib/perl5/site_perl/' to search path.

[Alex Vong]

[-- Attachment #1.1: Type: text/plain, Size: 61 bytes --]

Hello,

This patch should fix <https://bugs.gnu.org/30641>.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-gnu-youtube-viewer-Add-out-lib-perl5-site_perl-to-se.patch --]
[-- Type: text/x-diff, Size: 2294 bytes --]

From 2ad112584b40754fadb8269b7d5880500abe0050 Mon Sep 17 00:00:00 2001
From: Alex Vong <alexvong1995@gmail.com>
Date: Fri, 2 Mar 2018 21:22:59 +0800
Subject: [PATCH] gnu: youtube-viewer: Add '$out/lib/perl5/site_perl/' to
 search path.

Fixes <https://bugs.gnu.org/30671>.
Reported by Ricardo Wurmus <rekado@elephly.net>.

* gnu/packages/video.scm (youtube-viewer)[arguments]: In 'wrap-program'
phase, add '$out/lib/perl5/site_perl/' to search path.
---
 gnu/packages/video.scm | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 8190b0ffc..74dd290a3 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -1344,16 +1344,18 @@ audio, images) from the Web.  It can use either mpv or vlc for playback.")
                   (guix build utils)
                   (srfi srfi-26))
        #:module-build-flags '("--gtk")
-       #:phases (modify-phases %standard-phases
-                  (add-after 'install 'wrap-program
-                    (lambda* (#:key outputs #:allow-other-keys)
-                      (let ((bin-dir (string-append (assoc-ref outputs "out")
-                                                    "/bin/"))
-                            (perl-path (getenv "PERL5LIB")))
-                        (for-each (cut wrap-program <>
-                                       `("PERL5LIB" ":" prefix (,perl-path)))
-                                  (find-files bin-dir))
-                        #t))))))
+       #:phases
+       (modify-phases %standard-phases
+         (add-after 'install 'wrap-program
+           (lambda* (#:key outputs #:allow-other-keys)
+             (let* ((out (assoc-ref outputs "out"))
+                    (bin-dir (string-append out "/bin/"))
+                    (site-dir (string-append out "/lib/perl5/site_perl/"))
+                    (lib-path (getenv "PERL5LIB")))
+               (for-each (cut wrap-program <>
+                              `("PERL5LIB" ":" prefix (,lib-path ,site-dir)))
+                         (find-files bin-dir))
+               #t))))))
     (synopsis
      "Lightweight application for searching and streaming videos from YouTube")
     (description
-- 
2.16.2


[-- Attachment #1.3: Type: text/plain, Size: 14 bytes --]


Cheers,
Alex

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[bug#30677] [PATCH] gnu: youtube-viewer: Add '$out/lib/perl5/site_perl/' to search path.

[Alex Vong]

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

Alex Vong <alexvong1995@gmail.com> writes:

> Hello,
>
> This patch should fix <https://bugs.gnu.org/30641>.
>
Sorry, I mean <https://debbugs.gnu.org/30677>.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[bug#30677] [PATCH] gnu: youtube-viewer: Add '$out/lib/perl5/site_perl/' to search path.

[Alex Vong]

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

Alex Vong <alexvong1995@gmail.com> writes:

> Alex Vong <alexvong1995@gmail.com> writes:
>
>> Hello,
>>
>> This patch should fix <https://bugs.gnu.org/30641>.
>>
> Sorry, I mean <https://debbugs.gnu.org/30677>.

In fact, it should be <https://debbugs.gnu.org/30671>.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[bug#30677] [PATCH] gnu: youtube-viewer: Add '$out/lib/perl5/site_perl/' to search path.

[Ludovic Courtès]

Alex Vong <alexvong1995@gmail.com> skribis:

> From 2ad112584b40754fadb8269b7d5880500abe0050 Mon Sep 17 00:00:00 2001
> From: Alex Vong <alexvong1995@gmail.com>
> Date: Fri, 2 Mar 2018 21:22:59 +0800
> Subject: [PATCH] gnu: youtube-viewer: Add '$out/lib/perl5/site_perl/' to
>  search path.
>
> Fixes <https://bugs.gnu.org/30671>.
> Reported by Ricardo Wurmus <rekado@elephly.net>.
>
> * gnu/packages/video.scm (youtube-viewer)[arguments]: In 'wrap-program'
> phase, add '$out/lib/perl5/site_perl/' to search path.

Applied, thanks!

Ludo’.