Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Dominik George
Hi, On Sat, Sep 28, 2013 at 12:28:48AM -0500, Javier P.L. wrote: Package: wnpp Severity: wishlist Owner: Javier P.L. chilic...@ubuntu.com Very cool, I wanted to write such a frontend myself recently and found it blocked by the Google API Python bindings not being in Debian and me not wanting

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Javier P.L.
Hi, Yeah, some test would be great, you can grab the .dsc package from: http://mentors.debian.net/package/yt Or some precompiled .deb packages from: https://launchpad.net/~chilicuil/+archive/yt/+packages The raw .deb package should give not big troubles... Best regards On 28/09/13

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Julian Taylor
On 28.09.2013 07:28, Javier P.L. wrote: Package: wnpp Severity: wishlist Owner: Javier P.L. chilic...@ubuntu.com * Package name: yt Version : 2013.09.28 Upstream Author : Rich Wareham rj...@cantab.net * URL : https://github.com/rjw57/yt * License : MIT

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Dominik George
Hi, the package has apperaed now. Here are my remarks, some have already been mentioned, others are new. I have attached a patch that incorporates the necessary changes. - Do not use only the date for your package version! If you package a git master, use a version that sorts just below any

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Dominik George
Hi, you might want to test get-orig-source to update to bee8fd44c0, which incorporates a bugfix I committed upstream while testing your package ☺. -nik -- burny Ein Jabber-Account, sie alle zu finden; ins Dunkel zu treiben und ewig zu binden; im NaturalNet, wo die Schatten droh'n ;)!

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Dominik George
Hi, I'd also upgrade the Recommends to Depends. I do not see what use your package could be without youtube-dl and mplayer. I'd also change it to: Depends: youtube-dl, mplayer2 | mplayer Recommending mplayer2 is good, but having mplayer installed is sufficient as well. Note that mplayer2

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Dominik George
Attached is a patch that renames all relevant parts to youtube-cli (why didn't upstream stick with whitey, in the first place?) and implements the Depends I suggested. You could use it for the new upstream version. I am attaching all files relevant for the new upstream version. The patch needs to

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Javier P.L.
Hello Dominik, You're awesome!, I've applied most of your changes: - Do not use only the date for your package version! - I've applied your scheme, update it to bee8fd44c0 - omxplayer is not in Debian, so do not install the pi-yt script. - Applied - Install docs; NEWS.txt is renamed to

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Javier P.L.
The new package is at: http://mentors.debian.net/package/youtube-cli -- To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/20130928203320.gb3...@sup.lan

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Dominik George
Hi, - I've added mplayer2|mplayer to Depends, youtube-dl is still as Recommends the reason for this is that I've seen a lot of people downloading the youtube-dl script directly (including me), and I don't want to force them to use the debian youtube-dl package

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Dominik George
Hi, The new package is at: http://mentors.debian.net/package/youtube-cli I have reviewed it, and deemed the changes to README.source necessary. Also, as explained above, your consideration for local copies of youtube-dl contradict the Debian way. Attached is a patch that finalizes your

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-28 Thread Javier P.L.
Hi, On 28/09/13 at 10:40pm, Dominik George wrote: Hi, - I've added mplayer2|mplayer to Depends, youtube-dl is still as Recommends the reason for this is that I've seen a lot of people downloading the youtube-dl script directly (including me), and I don't want to

Bug#724801: ITP: yt -- yt is a command-line front-end to YouTube

2013-09-27 Thread Javier P.L.
Package: wnpp Severity: wishlist Owner: Javier P.L. chilic...@ubuntu.com * Package name: yt Version : 2013.09.28 Upstream Author : Rich Wareham rj...@cantab.net * URL : https://github.com/rjw57/yt * License : MIT Programming Lang: Python Description :