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

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Marius Bakke <mbakke <at> fastmail.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#30598: closed ([PATCH 0/5] gnu: Add youtube-viewer.)
Date: Wed, 28 Feb 2018 16:08:01 +0000

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

Your message dated Wed, 28 Feb 2018 17:07:44 +0100
with message-id <877eqxumgf.fsf <at> fastmail.com>
and subject line Re: [bug#30598] [PATCH 0/5] gnu: Add youtube-viewer.
has caused the debbugs.gnu.org bug report #30598,
regarding [PATCH 0/5] gnu: Add youtube-viewer.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
30598: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=30598
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Alex Vong <alexvong1995 <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: alexvong1995 <at> gmail.com
Subject: [PATCH 0/5] gnu: Add youtube-viewer.
Date: Sun, 25 Feb 2018 07:27:20 +0800

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

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.

Cheers,
Alex

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

[Message part 5 (message/rfc822, inline)]

From: Marius Bakke <mbakke <at> fastmail.com>
To: Alex Vong <alexvong1995 <at> gmail.com>
Cc: alexvong1995 <at> gmail.com, 30598-done <at> debbugs.gnu.org
Subject: Re: [bug#30598] [PATCH 0/5] gnu: Add youtube-viewer.
Date: Wed, 28 Feb 2018 17:07:44 +0100

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

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

> Marius Bakke <mbakke <at> fastmail.com> writes:
>
>> 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! :-)
>
> (I messed up my mail settings, so I am re-sending.)
>
> The next patch series should fix everything you've mentioned so far. I
> don't understand why wrap-program is used before but now I do.

Excellent, thank you!  I moved the perl-cairo and perl-pango packages to
gtk.scm as well and pushed the series as b0a2c9cbe..6f45e45d4.

> Thanks for your review also!

:-)

[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.