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
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 you
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 packag
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 C
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
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
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 provid
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
--
Ein Jabber-Account, sie alle zu finden; ins Dunkel zu treiben
und ewig zu binden; im NaturalNet, wo die Schatten droh'n ;)!
PGP-Fi
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
On 28.09.2013 07:28, Javier P.L. wrote:
> Package: wnpp
> Severity: wishlist
> Owner: "Javier P.L."
>
> * Package name: yt
> Version : 2013.09.28
> Upstream Author : Rich Wareham
> * URL : https://github.com/rjw57/yt
> * License : MIT
> Programming Lang: Pyt
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 at
Hi,
On Sat, Sep 28, 2013 at 12:28:48AM -0500, Javier P.L. wrote:
> Package: wnpp
> Severity: wishlist
> Owner: "Javier P.L."
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 to package it. O
Package: wnpp
Severity: wishlist
Owner: "Javier P.L."
* Package name: yt
Version : 2013.09.28
Upstream Author : Rich Wareham
* URL : https://github.com/rjw57/yt
* License : MIT
Programming Lang: Python
Description : yt is a command-line front-end to Yo
13 matches
Mail list logo
