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. 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.
signature.asc
Description: PGP signature
