* lisp/progmodes/python.el (python-shell-completion-setup-code): Use

__builtin__ module (or builtins in Python 3) and catch all errors
when importing readline and rlcompleter.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 5947c76ac17a5e186356fee1e730f8d16649e540..af75f8db3682f4b34c8e07ace0465396f9cf44fa 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,9 @@
+2014-11-27  Fabián Ezequiel Gallina  <fgallina@gnu.org>
+
+       * progmodes/python.el (python-shell-completion-setup-code): Use
+       __builtin__ module (or builtins in Python 3) and catch all errors
+       when importing readline and rlcompleter.
+
 2014-11-26  Stephen Berman  <stephen.berman@gmx.net>
 
        * calendar/todo-mode.el: Handle calling revert-buffer (bug#19187).

diff --git a/lisp/progmodes/python.el b/lisp/progmodes/python.el
index 4c27136f3b59029c6e4a187f5a342ab49b72eaef..48d80b99c6a02acb513b814c46b047ca20c070b9 100644 (file)
--- a/lisp/progmodes/python.el
+++ b/lisp/progmodes/python.el
@@ -2653,25 +2653,30 @@ This function takes the list of setup code to send from the
 
 (defcustom python-shell-completion-setup-code
   "try:
-    import readline, rlcompleter
+    import __builtin__
 except ImportError:
+    # Python 3
+    import builtins as __builtin__
+try:
+    import readline, rlcompleter
+except:
     def __PYTHON_EL_get_completions(text):
         return []
 else:
     def __PYTHON_EL_get_completions(text):
+        builtins = dir(__builtin__)
         completions = []
         try:
             splits = text.split()
             is_module = splits and splits[0] in ('from', 'import')
-            is_ipython = getattr(
-                __builtins__, '__IPYTHON__',
-                getattr(__builtins__, '__IPYTHON__active', False))
+            is_ipython = ('__IPYTHON__' in builtins or
+                          '__IPYTHON__active' in builtins)
             if is_module:
                 from IPython.core.completerlib import module_completion
                 completions = module_completion(text.strip())
-            elif is_ipython and getattr(__builtins__, '__IP', None):
+            elif is_ipython and '__IP' in builtins:
                 completions = __IP.complete(text)
-            elif is_ipython and getattr(__builtins__, 'get_ipython', None):
+            elif is_ipython and 'get_ipython' in builtins:
                 completions = get_ipython().Completer.all_completions(text)
             else:
                 i = 0