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

Ensure anaconda_mode can import jsonrpc, jedi #49

Merged
merged 2 commits into from
Nov 18, 2014

Conversation

tdavis
Copy link
Contributor

@tdavis tdavis commented Nov 18, 2014

If sys.path does not include an empty string, Python won't look for modules in the same directory as anaconda_mode.py. It isn't terribly uncommon for build tools/development environments to specify sys.path in a way that prevents such imports; for compatibility-sake, just make sure the empty string is present.

If ``sys.path`` does not include an empty string, Python won't look for modules in the same directory as ``anaconda_mode.py``. It isn't terribly uncommon for build tools/development environments to specify ``sys.path`` in a way that prevents such imports; for compatibility-sake, just make sure the empty string is present.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.11%) when pulling 94fbf77 on tdavis:patch-1 into 93c878b on proofit404:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.11%) when pulling 94fbf77 on tdavis:patch-1 into 93c878b on proofit404:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.11%) when pulling 94fbf77 on tdavis:patch-1 into 93c878b on proofit404:master.

@proofit404
Copy link
Contributor

I never meet such kind of restrictions my self. Just in case of my interest can you provide me any example? Thanks for this fix!!

proofit404 added a commit that referenced this pull request Nov 18, 2014
Ensure anaconda_mode can import jsonrpc, jedi
@proofit404 proofit404 merged commit 5543c95 into pythonic-emacs:master Nov 18, 2014
@tdavis
Copy link
Contributor Author

tdavis commented Nov 18, 2014

Sure thing. I am using Buildout in a project and configured the interpreter such that any script run with the project interpreter will use Buildout's egg directory as sys.path, instead of any other values. I have also configured some hooks into anaconda-mode to look for the project-relative interpreter. That interpreter isn't supposed to be used outside the project, therefor it does not include CWD in sys.path.

@proofit404
Copy link
Contributor

Your comment clarify situation for me. Thanks once again.

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.

3 participants