[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] gnu: Add python-pafy.
From: |
ng0 |
Subject: |
Re: [PATCH 1/2] gnu: Add python-pafy. |
Date: |
Fri, 04 Nov 2016 18:52:50 +0000 |
Hartmut Goebel <address@hidden> writes:
> Am 04.11.2016 um 19:18 schrieb ng0:
>> Hartmut Goebel <address@hidden> writes:
>>
>>> Am 02.11.2016 um 11:40 schrieb ng0:
>>>> + (inputs
>>>> + `(("youtube-dl" ,youtube-dl)))
>>> This needs to be a propagated input since python-pafy is a package itself.
>> Can you be more specific? I know now that other python inputs
>> should be propagated, but why non-python software? Or is
>> youtube-dl just another python software?
>
> Oh, sorry, I thought you know that youtube-dl is a python tool. (I not
> it since I'm using it myself).
Ah, thanks. I use it, but I have forgotten which language it is
written in as it is not relevant for me.
>
> Basically you were right: tools do not need to be propagated (at least
> as far as I understood).
>
> But the reason this needs to be propagated is: it is not used as a tool
> (via some system-call or such), but the python package is imported, see
> <https://github.com/mps-youtube/pafy/blob/develop/pafy/backend_youtube_dl.py#L11>.
> I have to admit that this is not obvious.
Ok, thanks for the explanation. I will add this with a comment to
a new revision of the patch series.
- [PATCH] mps-youtube, ng0, 2016/11/02
- [PATCH 2/2] gnu: Add mps-youtube., ng0, 2016/11/02
- [PATCH 1/2] gnu: Add python-pafy., ng0, 2016/11/02
- Re: [PATCH 1/2] gnu: Add python-pafy., Hartmut Goebel, 2016/11/02
- Re: [PATCH 1/2] gnu: Add python-pafy., ng0, 2016/11/04
- Re: [PATCH 1/2] gnu: Add python-pafy., Hartmut Goebel, 2016/11/04
- Re: [PATCH 1/2] gnu: Add python-pafy.,
ng0 <=
- [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