From: Brice Waegeneire <brice@waegenei.re>
To: me@tobias.gr, 40060@debbugs.gnu.org
Subject: [bug#40060] [PATCH 1/2] gnu: youtube-dl: Use ffmpeg and pycryptodome
Date: Tue, 17 Mar 2020 09:06:57 +0000 [thread overview]
Message-ID: <a61c16526f3dcdb95b63e4c3b01e2e49@waegenei.re> (raw)
In-Reply-To: <87y2s2lgxl.fsf@nckx>
On 2020-03-14 22:17, Tobias Geerinckx-Rice via Guix-patches via wrote:
> Brice Waegeneire 写道:
>> WARNING: You have requested multiple formats but ffmpeg or avconv are
>> not installed. The formats won't be merged.
NOTE: This message appear when using a format option like this
='bestvideo[height<=320]+bestaudio'=.
> This message is one of the best I've seen. It clearly explains to the
> user what's (not) going to happen, and what they can do to change that
> *if* they want to. Hence I think adding ffmpeg as a hard dependency
> is incorrect.
What about this one?
#+begin_src sh
$ youtue-dl https://www.youtube.com/watch\?v\=dp8PhLsUcFE
[youtube] dp8PhLsUcFE: Downloading webpage
[youtube] dp8PhLsUcFE: Downloading m3u8 information
[youtube] dp8PhLsUcFE: Downloading MPD manifest
[download] Destination: Bloomberg Global Financial News-dp8PhLsUcFE.mp4
ERROR: m3u8 download detected but ffmpeg or avconv could not be found.
Please install one.
#+end_src
When downloading a live stream from youtube.com (and proabably others),
~youtube-dl~ needs ffmpeg to download the HLS stream – I tried with just
~pycryptodome~ and it doesn't work.
> (I'd also oppose a youtube-dl-full variant, by the way. Packages
> aren't the right place for this; profiles are.)
>
> Does youtube-dl print a similarly clear message when pycryptodome is
> needed but missing? If not, that addition LGTM with a
>
> ("pycryptodome" ,pycryptodome) ; for the hlsnative downloader
#+begin_src sh
$ youtube-dl http://www.ivi.ru/watch/146500
[ivi] 146500: Downloading video JSON
ERROR: pycryptodomex not found. Please install it.
#+end_src
#+begin_src python
self.report_error('pycrypto not found. Please install
it.')
#+end_src
I digged a little deeper about this dependency, ~pycryptodome~
replace the deprecated ~pycrypto~ library with the same name space
(=Crypto=) while ~pycryptodomex~ uses it's own name space (=Cryptodome=)
to
provide similar functionality.
#+begin_src python
self.report_warning(
'hlsnative has detected features it does not support, '
'extraction will be delegated to ffmpeg')
#+end_src
So I'm not sure if adding ~pycryptodome(x)~ is that useful after all.
Especially if it can't totally replace ~ffmpeg~ when donating HLS
streams.
> comment. Cover letters & commit messages age badly.
Noted.
I think ~ffmpeg~ should be an input of ~youtube-dl~ while
~pycrytodome(x)~
don't need to since most (except the ivi downloader it seems)
functionality
can be achieved with ~ffmpeg~. WDYT?
next prev parent reply other threads:[~2020-03-17 9:08 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-14 14:34 [bug#40060] [PATCH 0/2] youtube-dl add ffmpeg, pycryptodome and zsh-completion Brice Waegeneire
2020-03-14 14:42 ` [bug#40060] [PATCH 1/2] gnu: youtube-dl: Use ffmpeg and pycryptodome Brice Waegeneire
2020-03-14 17:21 ` Leo Famulari
2020-03-14 17:39 ` Brice Waegeneire
2020-03-14 17:47 ` Leo Famulari
2020-03-14 22:20 ` Tobias Geerinckx-Rice via Guix-patches via
2020-03-14 22:17 ` Tobias Geerinckx-Rice via Guix-patches via
2020-03-15 17:25 ` Leo Famulari
2020-03-17 9:06 ` Brice Waegeneire [this message]
2020-03-14 14:42 ` [bug#40060] [PATCH 2/2] gnu: youtube-dl: Add zsh completion Brice Waegeneire
2020-03-23 17:15 ` [bug#40060] [PATCH v2 0/2] youtube-dl add ffmpeg and zsh-completion Brice Waegeneire
2020-03-23 17:15 ` [bug#40060] [PATCH v2 1/2] gnu: youtube-dl: Add 'ffmpeg' as input Brice Waegeneire
2020-03-23 17:15 ` [bug#40060] [PATCH v2 2/2] gnu: youtube-dl: Add zsh completion Brice Waegeneire
2020-03-23 19:28 ` [bug#40060] [PATCH v2 0/2] youtube-dl add ffmpeg and zsh-completion Mathieu Othacehe
2020-03-23 20:23 ` Brice Waegeneire
2020-03-23 20:43 ` Mathieu Othacehe
2020-04-01 13:01 ` [bug#40060] [PATCH v3] gnu: youtube-dl: Add 'ffmpeg' as input Brice Waegeneire
2020-04-01 13:40 ` Tobias Geerinckx-Rice via Guix-patches via
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=a61c16526f3dcdb95b63e4c3b01e2e49@waegenei.re \
--to=brice@waegenei.re \
--cc=40060@debbugs.gnu.org \
--cc=me@tobias.gr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).