GNU bug report logs - #30598
[PATCH 0/5] gnu: Add youtube-viewer.

Previous Next

Full log

Message #26 received at 30598 <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <mbakke <at> fastmail.com>
To: Alex Vong <alexvong1995 <at> gmail.com>, 30598 <at> debbugs.gnu.org
Cc: alexvong1995 <at> gmail.com
Subject: Re: [bug#30598] [PATCH 0/5] gnu: Add youtube-viewer.
Date: Mon, 26 Feb 2018 02:03:20 +0100

[Message part 1 (text/plain, inline)]

Alex Vong <alexvong1995 <at> gmail.com> writes:

> Tags: patch
>
> Hello,
>
> This patch series adds youtube-viewer and its dependencies. It can be
> used for searching youtube videos without running javascript. It also
> plays nice with tor which is a plus.

Thanks for this series!  The patches LGTM, but can you move the perl
packages to gtk.scm and glib.scm respectively?  That seems more
appropriate, as we already have e.g. pygtk in gtk.scm, and also avoids
some top-module cross-references.

(Additionally, perl.scm is huge, so it's good to spread the load to
appease the Guile compiler)

Thanks in advance! :-)

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.