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

Recursive shell call bug #23

Merged
merged 3 commits into from Sep 25, 2014
Merged

Recursive shell call bug #23

merged 3 commits into from Sep 25, 2014

Conversation

rocky
Copy link
Contributor

@rocky rocky commented Sep 22, 2014

When installing (not in a virtual environment), I have been getting:

RuntimeError: maximum recursion depth exceeded while calling a Python object
  File "/usr/local/lib/python2.7/dist-packages/solvebio-1.5.1-py2.7.egg/solvebio/cli/main.py", line 83, in parse_args
    return super(SolveArgumentParser, self).parse_args(args, namespace)
   File "/usr/lib/python2.7/dist-packages/argparse.py", line 1698, in parse_args
     args, argv = self.parse_known_args(args, namespace)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1730, in parse_known_args
     return self._parse_known_args(args, namespace)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1935, in _parse_known_args
    stop_index = consume_positionals(start_index)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1891, in consume_positionals
    take_action(action, args)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1800, in take_action
    action(self, namespace, argument_values, option_string)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1117, in __call__
    parser.parse_args(arg_strings, namespace)
  File "/usr/local/lib/python2.7/dist-packages/solvebio-1.5.1-py2.7.egg/solvebio/cli/main.py", line 83, in parse_args
    return super(SolveArgumentParser, self).parse_args(args, namespace)

This addresses that, although I'm not totally sure what's going on or if this is the best fix.

davecap and others added 3 commits September 18, 2014 10:42
RuntimeError: maximum recursion depth exceeded while calling a Python object
  File "/usr/local/lib/python2.7/dist-packages/solvebio-1.5.1-py2.7.egg/solvebio/cli/main.py", line 83, in parse_args
    return super(SolveArgumentParser, self).parse_args(args, namespace)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1698, in parse_args
    args, argv = self.parse_known_args(args, namespace)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1730, in parse_known_args
    return self._parse_known_args(args, namespace)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1935, in _parse_known_args
    stop_index = consume_positionals(start_index)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1891, in consume_positionals
    take_action(action, args)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1800, in take_action
    action(self, namespace, argument_values, option_string)
  File "/usr/lib/python2.7/dist-packages/argparse.py", line 1117, in __call__
    parser.parse_args(arg_strings, namespace)
  File "/usr/local/lib/python2.7/dist-packages/solvebio-1.5.1-py2.7.egg/solvebio/cli/main.py", line 83, in parse_args
    return super(SolveArgumentParser, self).parse_args(args, namespace)

This addresses that, although I'm not totally sure what's going on or if this is the best fix.
@pgeez
Copy link
Contributor

pgeez commented Sep 22, 2014

@rocky: does this show up when you install in a virtual env?

can you please clarify the issue at hand. do you see this issue when you 'pip install solvebio'?

@rocky
Copy link
Contributor Author

rocky commented Sep 22, 2014

WIll try later. The other time I had this, Cap helped me work around this by doing something with virtualwrapper. Why do you ask?

davecap added a commit that referenced this pull request Sep 25, 2014
@davecap davecap merged commit 9ab5286 into dev Sep 25, 2014
@rocky rocky deleted the recursive-shell-call-bug branch September 29, 2014 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants