guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] mps-youtube


From: Marius Bakke
Subject: Re: [PATCH] mps-youtube
Date: Wed, 02 Nov 2016 12:34:17 +0000
User-agent: Notmuch/0.23.1 (https://notmuchmail.org) Emacs/25.1.1 (x86_64-unknown-linux-gnu)

ng0 <address@hidden> writes:

> I am not sure about how the non-python runtime dependencies are supposed to 
> be handled in python packages.
>
> python-pafy needs youtube-dl to function.
> mps-youtube needs at least mpv and optionally ffmpeg to deal with conversion 
> of formats.

Have you checked whether the inputs are referenced in `guix gc -R
/gnu/store/...item`? If not, we'll need to make it record the absolute
paths of the dependencies somehow. Often this can be done with
'--with-foo=(string-append (assoc-ref inputs "foo") "/bin/foo"' in the
configure step. Another approach is to substitute those commands
with the full path directly in the code.

> So far they are in (inputs). Are those supposed to be installed by the users 
> themselves?

For optional dependencies (i.e. if mps-youtube gracefully handles
missing ffmpeg), leaving it up to the user is good (but then it should
not be an input). For "hard" runtime dependencies, the absolute paths
must be recorded so that they won't get garbage collected.

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]