View previous topic :: View next topic |
Author |
Message |
Irre Guru
Joined: 09 Nov 2013 Posts: 434 Location: Stockholm
|
Posted: Sun Jun 28, 2015 8:18 am Post subject: [solved]Newer youtube-dl versions do not work for me |
|
|
They "compile" with errors during install. Code: | ..
>>> Install youtube-dl-2015.06.25 into /var/tmp/portage/net-misc/youtube-dl-2015.06.25/image/ category net-misc
Compiling /var/tmp/portage/net-misc/youtube-dl-2015.06.25/image///usr/lib/python2.7/site-packages/youtube_dl/extractor/__init__.py ...
Sorry: IndentationError: unexpected indent (__init__.py, line 666)
Compiling /var/tmp/portage/net-misc/youtube-dl-2015.06.25/image///usr/lib/python2.7/site-packages/youtube_dl/extractor/__init__.py ...
Sorry: IndentationError: unexpected indent (__init__.py, line 666)
* Fixing shebang in usr/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py.
* Fixing shebang in usr/lib/python2.7/site-packages/youtube_dl/utils.py.
* Fixing shebang in usr/lib/python2.7/site-packages/youtube_dl/__main__.py.
* Fixing shebang in usr/lib/python2.7/site-packages/youtube_dl/__init__.py.
* Fixing shebang in usr/bin/youtube-dl.
>>> Completed installing youtube-dl-2015.06.25 into /var/tmp/portage/net-misc/youtube-dl-2015.06.25/image/
.. |
When running I get this error: Code: | youtube-dl -i -o test.mp4 http://www.zdf.de/ZDFmediathek/hauptnavigation/startseite/#/beitrag/video/2408478/Kalaschnikow-und-Kalifat
Traceback (most recent call last):
File "/usr/bin/youtube-dl", line 3, in <module>
import youtube_dl
File "/usr/lib/python2.7/site-packages/youtube_dl/__init__.py", line 43, in <module>
from .extractor import gen_extractors, list_extractors
File "/usr/lib/python2.7/site-packages/youtube_dl/extractor/__init__.py", line 666
XHamsterIE,
^
IndentationError: unexpected indent |
This version compiles, installs and works fine: net-misc/youtube-dl-2015.04.03
Last edited by Irre on Sun Jul 05, 2015 5:23 pm; edited 1 time in total |
|
Back to top |
|
|
Irre Guru
Joined: 09 Nov 2013 Posts: 434 Location: Stockholm
|
Posted: Sun Jul 05, 2015 6:51 am Post subject: |
|
|
The error still remain in new version during installation:
>>> Install youtube-dl-2015.07.04 into /var/tmp/portage/net-misc/youtube-dl-2015.07.04/image/ category net-misc
Compiling /var/tmp/portage/net-misc/youtube-dl-2015.07.04/image///usr/lib/python2.7/site-packages/youtube_dl/extractor/__init__.py ...
Sorry: IndentationError: unexpected indent (__init__.py, line 547)
Compiling /var/tmp/portage/net-misc/youtube-dl-2015.07.04/image///usr/lib/python2.7/site-packages/youtube_dl/extractor/__init__.py ...
Sorry: IndentationError: unexpected indent (__init__.py, line 547)
/usr/lib/python2.7/site-packages/youtube_dl/extractor/__init__.py:
...
from .tmz import (
TMZIE,
TMZArticleIE,
)
TNAFlixIE,
EMPFlixIE,
MovieFapIE,
)
...
I don't understand python, but it looks strange. Where can I report this error? Removing the code above "solves" the problem for me. |
|
Back to top |
|
|
idella4 Retired Dev
Joined: 09 Jun 2006 Posts: 1600 Location: Australia, Perth
|
Posted: Sun Jul 05, 2015 7:23 am Post subject: |
|
|
Irre wrote: | The error still remain in new version during installation:
/usr/lib/python2.7/site-packages/youtube_dl/extractor/__init__.py:
...
from .tmz import (
TMZIE,
TMZArticleIE,
)
TNAFlixIE,
EMPFlixIE,
MovieFapIE,
)
...
I don't understand python, but it looks strange. Where can I report this error? Removing the code above "solves" the problem for me. |
Not yet having perused this __init__.py, it's possible that the upstream makers introduced an error by way of a typo.
The above appears to have 7 spaces from ^ to the first char in the func.
TMZArticleIE,
)
looks like it is an accidental line break and should be on one line
Removing the code could mean anything to me, far too general.
The norm for python is to increase the indent by 4 spaces and python is unforgiving about it.
If I were investigating, I'd look at issues or bugs filed upstream and see if this is known. Get it from the ebuild.
You can take the easy way and just file it under gentoo bugzilla and I'll have to do it anyway.
Once you find a corrected format that works the upstream are all but obliged to patch it to match or an equal. _________________ idella4@aus |
|
Back to top |
|
|
charles17 Advocate
Joined: 02 Mar 2008 Posts: 3664
|
Posted: Sun Jul 05, 2015 9:01 am Post subject: |
|
|
Irre wrote: | Where can I report this error? Removing the code above "solves" the problem for me. |
https://bugs.gentoo.org/ |
|
Back to top |
|
|
Irre Guru
Joined: 09 Nov 2013 Posts: 434 Location: Stockholm
|
Posted: Sun Jul 05, 2015 9:06 am Post subject: |
|
|
I run this last version on arch-linux with no errors. I try to file a bug report.. |
|
Back to top |
|
|
idella4 Retired Dev
Joined: 09 Jun 2006 Posts: 1600 Location: Australia, Perth
|
|
Back to top |
|
|
Irre Guru
Joined: 09 Nov 2013 Posts: 434 Location: Stockholm
|
Posted: Sun Jul 05, 2015 12:50 pm Post subject: |
|
|
idella4 wrote: | See if you can get someone to replicate it would assist the ascertion that it's actually a bug |
I can replicate it myself on quite different Gentoo installations: Raspberry PI ARM6 and x86_64 Intel
If you run: Code: | emerge -v1 --quiet-build n youtube-dl | you will se errors during install stage but they are unfortunately neglected. |
|
Back to top |
|
|
khayyam Watchman
Joined: 07 Jun 2012 Posts: 6227 Location: Room 101
|
Posted: Sun Jul 05, 2015 4:00 pm Post subject: Re: Newer youtube-dl versions do not work for me |
|
|
Irre wrote: | Code: | File "/usr/lib/python2.7/site-packages/youtube_dl/extractor/__init__.py", line 666
XHamsterIE,
^
IndentationError: unexpected indent |
|
Irre ... I'm pretty sure this issue is due to the ebuild, specifically the 'offensive' useflag. If you look at 'src_prepare()' xhamster has a sed "single line import statement" but the __init__.py for > youtube-dl-2015.06.25 has 'from .xhamster import' as multi-line. If I'm correct then setting the 'offensive' useflag should solve your issue (at least the 'if ! use offensive' won't apply ... and so the sed leaving the trailing lines will be avoided).
HTH & best ... khay |
|
Back to top |
|
|
Irre Guru
Joined: 09 Nov 2013 Posts: 434 Location: Stockholm
|
Posted: Sun Jul 05, 2015 5:22 pm Post subject: |
|
|
Thank you khayyam, "offensive" useflag solved everything. |
|
Back to top |
|
|
idella4 Retired Dev
Joined: 09 Jun 2006 Posts: 1600 Location: Australia, Perth
|
Posted: Mon Jul 06, 2015 1:45 am Post subject: |
|
|
I haven't resolved this myself but the outcome here is that the ebuild is broken, therefore https://bugs.gentoo.org/
is the place to file. The maintainer of this ebuild is senior and well informed but he seems to have missed something here.
(I haven't yet run it myself nor am I inclined) Also it's not purely becuse it's a pythonic package. It appears it will work with "USE=offensive",
so while setting IUSE 'offensive+' will ensure it will build, it is still expected to work with "USE=-offensive".
Either cite this forum post or summarise the findings from here in the bug and maintainer should have it fixed inside 24 hrs. _________________ idella4@aus |
|
Back to top |
|
|
kurly Apprentice
Joined: 02 Apr 2012 Posts: 260
|
Posted: Mon Jul 06, 2015 2:12 am Post subject: |
|
|
Smells like bug 553668. |
|
Back to top |
|
|
Roman_Gruber Advocate
Joined: 03 Oct 2006 Posts: 3846 Location: Austro Bavaria
|
Posted: Tue Jul 07, 2015 8:10 am Post subject: |
|
|
same issue: +1 on bug report here. with my output |
|
Back to top |
|
|
|