[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 14:20:42 +0000 |
User-agent: |
Notmuch/0.23.1 (https://notmuchmail.org) Emacs/25.1.1 (x86_64-unknown-linux-gnu) |
ng0 <address@hidden> writes:
>>> 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.
>>
>
> With the info I gave above, I'd say I remove the dependencies
> which are not found and add information to the description.
> For example: … it can be used with either mpv or mplayer, for
> optional format conversion you need to use ffmpeg.
>
> What do you think?
That sounds sensible. It seems to handle the case where those are
missing as well, which is good.
signature.asc
Description: PGP signature
- Re: [PATCH 1/2] gnu: Add python-pafy., (continued)
- Re: [PATCH 1/2] gnu: Add python-pafy., Hartmut Goebel, 2016/11/04
- Re: [PATCH 1/2] gnu: Add python-pafy., ng0, 2016/11/04
- [PATCH] mps-youtube, revision 3, ng0, 2016/11/04
- [PATCH 1/2] gnu: Add python-pafy., ng0, 2016/11/04
- [PATCH 2/2] gnu: Add mps-youtube., ng0, 2016/11/04
- Re: [PATCH] mps-youtube, revision 3, Hartmut Goebel, 2016/11/04
- Re: [PATCH] mps-youtube, revision 3, Marius Bakke, 2016/11/05
- Re: [PATCH] mps-youtube, revision 3, ng0, 2016/11/05
Re: [PATCH] mps-youtube, Marius Bakke, 2016/11/02