Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

- #10921

Closed
nakashbar opened this issue Oct 14, 2016 · 3 comments
Closed

- #10921

nakashbar opened this issue Oct 14, 2016 · 3 comments

Comments

@nakashbar
Copy link

@nakashbar nakashbar commented Oct 14, 2016

No description provided.

@yan12125
Copy link
Collaborator

@yan12125 yan12125 commented Oct 14, 2016

Could you run:

python -m youtube_dl -v https://www.youtube.com/watch?v=QI1ruAQypxc --write-pages

And zip all *.dump files and upload?

@urit
Copy link

@urit urit commented Oct 14, 2016

This is the fix:

def extract_object(self, objname):

    obj = {}
    obj_m = re.search(
        (r'(?:var\s+)?%s\s*=\s*\{' % re.escape(objname)) +
        r'\s*(?P<fields>([a-zA-Z$0-9]+\s*:\s*function\(.*?\)\s*\{.*?\}(?:,\s*)?)+)' +
        r'\}\s*;',
        self.code)
    fields = obj_m.group('fields')
    # Currently, it only supports function definitions
    fields_m = re.finditer(
        r'(?P<key>[a-zA-Z$0-9]+)\s*:\s*function'
        r'\((?P<args>[a-z,]+)\){(?P<code>[^}]+)}',
        fields)
    for f in fields_m:
        argnames = f.group('args').split(',')
        obj[f.group('key')] = self.build_function(argnames, f.group('code'))


    return obj
@dstftw
Copy link
Collaborator

@dstftw dstftw commented Oct 14, 2016

There is no such commit 5ddb0a4. You are using a modified version of youtube-dl. You must provide full output with --dump-pages with original youtube-dl code.

@dstftw dstftw closed this Oct 14, 2016
@nakashbar nakashbar changed the title Having problems playing songs Ver 2016.10.12 - Oct 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.