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

  • Done
  • quality assurance status badge
Details
2 participants
  • Alex Vong
  • Ludovic Courtès
Owner
unassigned
Submitted by
Alex Vong
Severity
normal
A
A
Alex Vong wrote on 2 Mar 2018 14:59
(address . guix-patches@gnu.org)(address . alexvong1995@gmail.com)
87woyuk28k.fsf@gmail.com
Hello,

This patch should fix https://bugs.gnu.org/30641.
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.

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(-)

Toggle diff (35 lines)
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
Cheers,
Alex
-----BEGIN PGP SIGNATURE-----

iHUEARYIAB0WIQTdXC2pN4kXBjiMDs8cGaIXrbrArQUCWplYqwAKCRAcGaIXrbrA
rd1xAP0WH5dlwvwesUtgxc40+8DYpO/1HoSvAHpYlFnDmUD0FwD/eA9oKL9tappR
taZvpLY18N3Jf4Y3mSk06zraoc3+jgo=
=Qy/j
-----END PGP SIGNATURE-----

A
A
Alex Vong wrote on 2 Mar 2018 15:03
(address . guix-patches@gnu.org)(address . alexvong1995@gmail.com)
87o9k6k218.fsf@gmail.com
Alex Vong <alexvong1995@gmail.com> writes:

Toggle quote (4 lines)
> Hello,
>
> This patch should fix <https://bugs.gnu.org/30641>.
>
-----BEGIN PGP SIGNATURE-----

iHUEARYIAB0WIQTdXC2pN4kXBjiMDs8cGaIXrbrArQUCWplZswAKCRAcGaIXrbrA
rWWZAQCoL8RmA8b0dXB0qaV7nraocgyl5JXD8T79wniAcUgXWQD+KiPGqk+20xiF
YqycoCWr6K/iUh5wVnJtbjcY3jO89QA=
=OqPr
-----END PGP SIGNATURE-----

A
A
Alex Vong wrote on 2 Mar 2018 15:08
(address . guix-patches@gnu.org)(address . alexvong1995@gmail.com)
87k1uuk1th.fsf@gmail.com
Alex Vong <alexvong1995@gmail.com> writes:

Toggle quote (8 lines)
> 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.
-----BEGIN PGP SIGNATURE-----

iHUEARYIAB0WIQTdXC2pN4kXBjiMDs8cGaIXrbrArQUCWplaygAKCRAcGaIXrbrA
rdCHAQCpF5oGC6h3tOjd1Q8FlVSFcq38IxhJUSw1vfJ2jkuJZQEA31eHEP2TM0ad
OFwrm3kgJ33J+VWCcPx6MkSlwo53Fgc=
=hFTO
-----END PGP SIGNATURE-----

L
L
Ludovic Courtès wrote on 7 Mar 2018 11:24
Re: [bug#30677] [PATCH] gnu: youtube-viewer: Add '$out/lib/perl5/site_perl/' to search path.
(name . Alex Vong)(address . alexvong1995@gmail.com)(address . 30677@debbugs.gnu.org)
87zi3ktc8p.fsf@gnu.org
Alex Vong <alexvong1995@gmail.com> skribis:

Toggle quote (12 lines)
> 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’.
L
L
Ludovic Courtès wrote on 7 Mar 2018 11:24
control message for bug #30677
(address . control@debbugs.gnu.org)
87y3j4tc8a.fsf@gnu.org
tags 30677 fixed
close 30677
?