GNU bug report logs -
#35033
[PATCH] gnu: mpv: Set absolute path to 'youtube-dl'.
Previous Next
Reported by: Timo Eisenmann <eisenmann <at> fn.de>
Date: Thu, 28 Mar 2019 19:05:01 UTC
Severity: normal
Tags: patch
Done: Ludovic Courtès <ludo <at> gnu.org>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 35033 in the body.
You can then email your comments to 35033 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#35033
; Package
guix-patches
.
(Thu, 28 Mar 2019 19:05:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Timo Eisenmann <eisenmann <at> fn.de>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Thu, 28 Mar 2019 19:05:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/video.scm (mpv)[arguments]: Add phase 'patch-paths' to
substitute in the absolute path to 'youtube-dl'.
---
gnu/packages/video.scm | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 55a1931530..3c5ade55a3 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -1326,6 +1326,13 @@ SVCD, DVD, 3ivx, DivX 3/4/5, WMV and H.264 movies.")
(arguments
'(#:phases
(modify-phases %standard-phases
+ (add-after
+ 'unpack 'patch-paths
+ (lambda* (#:key inputs #:allow-other-keys)
+ (let ((ytdl (assoc-ref inputs "youtube-dl")))
+ (substitute* "player/lua/ytdl_hook.lua"
+ (("\"youtube-dl\",")
+ (string-append "\"" ytdl "/bin/youtube-dl\","))))))
(add-before
'configure 'setup-waf
(lambda* (#:key inputs #:allow-other-keys)
--
2.21.0
Information forwarded
to
guix-patches <at> gnu.org
:
bug#35033
; Package
guix-patches
.
(Wed, 03 Apr 2019 20:25:01 GMT)
Full text and
rfc822 format available.
Message #8 received at 35033 <at> debbugs.gnu.org (full text, mbox):
Hi Timo,
Timo Eisenmann <eisenmann <at> fn.de> skribis:
> * gnu/packages/video.scm (mpv)[arguments]: Add phase 'patch-paths' to
> substitute in the absolute path to 'youtube-dl'.
I think it’s a case where it may be more appropriate to look for
‘youtube-dl’ in $PATH because ‘youtube-dl’ is updated frequently.
WDYT?
Thanks,
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#35033
; Package
guix-patches
.
(Wed, 03 Apr 2019 20:37:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 35033 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
We normally don't require the user to install dependencies such as
youtube-dl into the profile. I think we should retain that behaviour
even in this case with mpv. It would mean fewer surprises for the
user. If I'm guessing correctly, mpv doesn't require a lot of effort to
rebuild.
[signature.asc (application/pgp-signature, inline)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#35033
; Package
guix-patches
.
(Wed, 03 Apr 2019 22:52:02 GMT)
Full text and
rfc822 format available.
Message #14 received at 35033 <at> debbugs.gnu.org (full text, mbox):
On Wed, 03 Apr 2019 22:24:12 +0200
Ludovic Courtès <ludo <at> gnu.org> wrote:
> Hi Timo,
>
> Timo Eisenmann <eisenmann <at> fn.de> skribis:
>
> > * gnu/packages/video.scm (mpv)[arguments]: Add phase 'patch-paths'
> > to substitute in the absolute path to 'youtube-dl'.
>
> I think it’s a case where it may be more appropriate to look for
> ‘youtube-dl’ in $PATH because ‘youtube-dl’ is updated frequently.
>
> WDYT?
I think we should make mpv's youtube-dl feature work by default instead
of requiring the user to install youtube-dl into their profile.
That being said if, for some reason, someone does not want to update mpv
and still use a newer version of youtube-dl, they can do so by linking
the youtube-dl binary into mpv's config dir (by default: ~/.config/mpv).
>
> Thanks,
> Ludo’.
Reply sent
to
Ludovic Courtès <ludo <at> gnu.org>
:
You have taken responsibility.
(Mon, 08 Apr 2019 08:31:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Timo Eisenmann <eisenmann <at> fn.de>
:
bug acknowledged by developer.
(Mon, 08 Apr 2019 08:31:02 GMT)
Full text and
rfc822 format available.
Message #19 received at 35033-done <at> debbugs.gnu.org (full text, mbox):
Hello,
Timo Eisenmann <eisenmann <at> fn.de> skribis:
> On Wed, 03 Apr 2019 22:24:12 +0200
> Ludovic Courtès <ludo <at> gnu.org> wrote:
[...]
>> I think it’s a case where it may be more appropriate to look for
>> ‘youtube-dl’ in $PATH because ‘youtube-dl’ is updated frequently.
>>
>> WDYT?
>
> I think we should make mpv's youtube-dl feature work by default instead
> of requiring the user to install youtube-dl into their profile.
Arun Isaac <arunisaac <at> systemreboot.net> skribis:
> We normally don't require the user to install dependencies such as
> youtube-dl into the profile. I think we should retain that behaviour
> even in this case with mpv. It would mean fewer surprises for the
> user. If I'm guessing correctly, mpv doesn't require a lot of effort to
> rebuild.
Alright, that makes sense (and indeed, mpv builds very quickly.)
I added a #t at the end of the phase and committed.
Thank you!
Ludo’.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Mon, 06 May 2019 11:24:04 GMT)
Full text and
rfc822 format available.
This bug report was last modified 4 years and 356 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.