Bug 34394

Summary:	change depend on python3 module
Product:	Sisyphus	Reporter:	ildar <ildar>
Component:	youtube-dl	Assignee:	Gleb F-Malinovskiy <glebfm>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	cronbuild, glebfm
Version:	unstable
Hardware:	all
OS:	Linux

Description

ildar 2017-12-25 12:05:10 MSK

Вероятно, как минимум, /usr/bin/youtube-dl стОит нацеливать на python3, поскольку он лучше работает:
> $ youtube-dl https://music.yandex.ru/album/3321317
> ...
> Traceback (most recent call last):
>   File "/usr/bin/youtube-dl", line 9, in <module>
>     load_entry_point('youtube-dl==2017.5.23', 'console_scripts', 'youtube-dl')()
>   File "/usr/lib/python2.7/site-packages/youtube_dl/__init__.py", line 465, in m
> ain
>     _real_main(argv)
>   File "/usr/lib/python2.7/site-packages/youtube_dl/__init__.py", line 455, in _
> real_main
>     retcode = ydl.download(all_urls)
>   File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 1896, in
>  download
>     url, force_generic_extractor=self.params.get('force_generic_extractor', Fals
> e))
>   File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 771, in
> extract_info
>     return self.process_ie_result(ie_result, download, extra_info)
>   File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 962, in
> process_ie_result
>     extra_info=extra)
>   File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 832, in
> process_ie_result
>     extra_info=extra_info)
>   File "/usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 760, in
> extract_info
>     ie_result = ie.extract(url)
>   File "/usr/lib/python2.7/site-packages/youtube_dl/extractor/common.py", line 4
> 33, in extract
>     ie_result = self._real_extract(url)
>   File "/usr/lib/python2.7/site-packages/youtube_dl/extractor/yandexmusic.py", l
> ine 144, in _real_extract
>     return self._get_track_info(track)
>   File "/usr/lib/python2.7/site-packages/youtube_dl/extractor/yandexmusic.py", l
> ine 102, in _get_track_info
>     'url': self._get_track_url(track['storageDir'], track['id']),
>   File "/usr/lib/python2.7/site-packages/youtube_dl/extractor/yandexmusic.py", l
> ine 88, in _get_track_url
>     % (data['host'], key, data['ts'] + data['path'], storage[1]))
> TypeError: unsupported operand type(s) for +: 'int' and 'unicode'

При этом отработал нормально:
> $ python3 -m youtube_dl https://music.yandex.ru/album/3321317

Comment 1

Gleb F-Malinovskiy 2017-12-25 12:47:09 MSK

(In reply to comment #0)
> Вероятно, как минимум, /usr/bin/youtube-dl стОит нацеливать на python3,
> поскольку он лучше работает:

И так имеет смысл переключаться на трипитон, а если лучше работает, то точно.  Согласен.

Comment 2

ildar 2017-12-26 12:28:38 MSK

Глеб, новая версия в Вашем git не до конца исправлена, http://git.altlinux.org/people/glebfm/packages/?p=youtube-dl.git;a=blob;f=youtube-dl.spec;h=99a9ed93359797a237eb5d27fd481ac318540736;hb=1bcfb6e921efbd98b33c34ff34ddfcf517de2ef5#l16

Comment 3

Gleb F-Malinovskiy 2017-12-26 13:01:34 MSK

(In reply to comment #2)
> Глеб, новая версия в Вашем git не до конца исправлена,
> http://git.altlinux.org/people/glebfm/packages/?p=youtube-dl.git;a=blob;f=youtube-dl.spec;h=99a9ed93359797a237eb5d27fd481ac318540736;hb=1bcfb6e921efbd98b33c34ff34ddfcf517de2ef5#l16

Точно, спасибо.

Comment 4

Repository Robot 2017-12-28 22:17:59 MSK

youtube-dl-2017.12.23-alt1 -> sisyphus:

Tue Dec 26 2017 Gleb F-Malinovskiy <glebfm@altlinux> 2017.12.23-alt1
- Updated to 2017.12.23.
- Switched /usr/bin/youtube-dl to use python3 (ALT#34394).