Bug 34394 - change depend on python3 module
Summary: change depend on python3 module
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: youtube-dl (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Gleb F-Malinovskiy
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-25 12:05 MSK by ildar
Modified: 2017-12-28 22:17 MSK (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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).