* Package name : yt Version : 2013.09.28 Upstream Author : Rich Wareham <rjw57@cantab.net> * URL : https://github.com/rjw57/yt * License : MIT Programming Lang: Python Description : yt is a command-line front-end to YouTube yt is a command-line front-end to YouTube which allows you to browse YouTube videos and play them directly from the command-line. It uses youtube-dl and mplayer or omxplayer to actually play the videos
Hi, 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. On a first glance, yt does not need the API bindings. If you need a tester, please contact me.
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
yt is an awfully generic name for such a specialized application. can it be renamed e.g. youtube or something that is less likely to create namespace conflicts?
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
version number upstream might release in the future. Your version
string 2013.09.28 will supersede any upstream tag, should upstream
ever decide to introduce such. A good versioning scheme is
0~yyyymmdd+gitxxxxxxxxxx, e.g. for your package
0~20130928+git6e577736d0. You should remove the package from mentors
completely and reupload.
- omxplayer is not in Debian, so do not install the pi-yt script.
- Install docs; NEWS.txt is renamed to CHANGELOG.txt to satisfy lintian.
- Remove whitey egg-info.
- Do not just tar up a git clone to create an orig tgz! Use a proper
get-orig-source that repackages a Github snapshot. Once this is done,
you jsut need to do this to create a new orig tgz, even for a new
upstream version:
dch -i # insert proper new upstream version, we cannot use uscan
# here because we cannot really monitor upstream commits
debian/rules get-orig-source # drops yt_<UPSTREAM-VERSION>.orig.tar.xz
# in current directory
# unpack and move your Debian directory
# or use git-buildpackage
- As setup.py clearly states Apache 2.0, putthat dual license choice in
debian/copyright.
- I added myself to debian/copyright for the rules file, and to Uploaders
to justify the changelog entry. If you incorporate the changes and udpate
the changelog yourself, you can remvoe Uploaders if you want.
The package is lintian-clean now, except for the missing watch file,
which is ok.
HTH,
Nik
Hi, you might want to test get-orig-source to update to bee8fd44c0, which incorporates a bugfix I committed upstream while testing your package ☺.
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 provides mplayer, so the preceding mplayer2 prioritizes it if none is installed.
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 be applied on top the debian directory resulting from applying my last patch.
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 CHANGELOG.txt to satisfy lintian.
- Applied
- Remove whitey egg-info.
- Applied
- Do not just tar up a git clone to create an orig tgz! Use a proper
get-orig-source that repackages a Github snapshot. Once this is done,
you jsut need to do this to create a new orig tgz, even for a new
upstream version:
dch -i # insert proper new upstream version, we cannot use uscan
# here because we cannot really monitor upstream commits
debian/rules get-orig-source # drops yt_<UPSTREAM-VERSION>.orig.tar.xz
# in current directory
# unpack and move your Debian directory
# or use git-buildpackage
- I had no idea how to this, I've learned and applied your changes
- As setup.py clearly states Apache 2.0, putthat dual license choice in
debian/copyright.
- Applied
- I added myself to debian/copyright for the rules file, and to Uploaders
to justify the changelog entry. If you incorporate the changes and udpate
the changelog yourself, you can remvoe Uploaders if you want.
- Applied
- I'd also upgrade the Recommends to Depends. I do not see what use your
ackage could be without youtube-dl and mplayer. I'd also change it to:
Depends: youtube-dl, mplayer2 | mplayer
- 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 instead of his. People installing youtube-cli with the
--no-install-recommends option then can use his own youtube-dl version
- Attached is a patch that renames all relevant parts to youtube-cli
- Applied
I've decided to keep README.source, it makes no harm and I've an awful memory, if I
don't write this down, I'll forget it. I'm still not very experimented with debian
packaging.
It still has missing the watch file, I'll keep a look at upstream to see if at some point
he decides to use tags, then I'll be able to create a watch file.
Best regards
The new package is at: http://mentors.debian.net/package/youtube-cli
Hi, People deciding to use local copies of programs that are in Debian is clearly unsupported. I, and to my knowledge quite some other people, have Install-Recommends disabled by default, and this will break your package entirely. This said, youtube-cli *does* need youtube-dl. If someone decides to use their own copy, they can also use equivs or something. The Debian package system is about using Debian packages, and that's what we expect our users to do. Please remember updating it to the get-orig-source method. The method you describe in your original README.source is *invalid* as it creates a wrong orig tarball (containing .git/ and everything). The only valid method to fetch the source for your package now is debian/rules get-orig-source.
Hi, 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 packaging work, I think you can put up your package for sponsorship after applying it (if I were a DD, I would not upload it if you did not apply the patch). Fell free to mention my review if posting to debian-mentors. You can use the template supplied by mentors and reportbug to file an RFS bug that will also trigger mail to the list. Cheers, Nik
Hi, Thanks for your continue feedback, I've modified d/control to reflect the proposal target. Any additional comment is welcome! The new dsc file is at: http://mentors.debian.net/package/youtube-cli
retitle 724801 RFP: youtube-cli -- command-line client for YouTube' noowner 724801 tag 724801 - pending thanks Hi, A long time ago, you expressed interest in packaging youtube-cli. Unfortunately, it seems that it did not happen. In Debian, we try not to keep ITP bugs open for a too long time, as it might cause other prospective maintainers to refrain from packaging the software. This is an automatic email to change the status of youtube-cli from ITP (Intent to Package) to RFP (Request for Package), because this bug hasn't seen any activity during the last 12 months. If you are still interested in packaging youtube-cli, please send a mail to <control@bugs.debian.org> with: retitle 724801 ITP: youtube-cli -- command-line client for YouTube' owner 724801 ! thanks It is also a good idea to document your progress on this ITP from time to time, by mailing <724801@bugs.debian.org>. If you need guidance on how to package this software, please reply to this email, and/or contact the debian-mentors@lists.debian.org mailing list. Thank you for your interest in Debian,