It was the first time I used argparse I just followed an example, maybe try
uploading something with spaces on default, see if that breaks it. If
theres no issue with default I will make it default string
On Nov 20, 2013 7:57 PM, "Dennis Ideler" <notifications@github.com> wrote:

> In src/freeseer/framework/youtube.py:
>
> > +            parents=[tools.argparser])
> > +
> > +        self.default_video_directory = get_default_video_folder()
> > +
> > +        # Framework is being used as a command-line script pass along args
> > +        if argv is not None:
> > +            # Look for client secrets and OAuth2 token inside same folder as this
> > +            # script (src/freeseer/framework, we can default somewhere else
> > +            # later)
> > +            client_secrets = os.path.join(
> > +                os.path.dirname(__file__), 'client_secrets.json')
> > +
> > +            oauth2_token = os.path.join(
> > +                os.path.dirname(__file__), 'oauth2_token.json')
> > +
> > +            parser.add_argument("-c", "--client_secrets", type=unicode,
>
> What's the reason for using type=unicode instead of the default string?
> In a quick test I did, leaving the type as the default works with a path
> like "22少女時代22/client_secrets.json" while type=unicode doesn't.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Freeseer/freeseer/pull/402/files#r7812387>
> .
>