Marius Bakke <mba...@fastmail.com> writes: > Hartmut Goebel <h.goe...@crazy-compilers.com> writes: > >> Am 04.11.2016 um 20:23 schrieb ng0: >>> [PATCH 1/2] gnu: Add python-pafy. >>> >>> - made youtube-dl propagated-input as suggested by harmut >>> >>> [PATCH 2/2] gnu: Add mps-youtube. >>> >>> - no changes >>> >> >> Both LGTM. >> >> Adding the comment about youtube_dl was a good idea :-) > > Both of these packages have 'or later' in the source license header. > Also mps-youtube does not need mpv or ffmpeg in inputs, since they are > optional runtime dependencies.
Oh... I had forgotten about these two changes I wanted to make. Thanks for fixing it! > Additionally home-page should be https. > > Other than that this works great, neat program! > > I pushed the patches with these changes, and a minor edit to the > mps-youtube description to make it clear that mplayer or mpv is > required. >
