/pidgin/main: 7d134a4a87b2: Ensure all Python scripts are Py3k c...

Elliott Sales de Andrade <[email protected]> Tue, 06 Sep 2016 11:30:53 -0400
Newsgroups gmane.comp.gnome.gaim.cvs
Message-ID <[email protected]>
Changeset: 7d134a4a87b2b21966b4b3ffda36136d9f040666
Author:	 Elliott Sales de Andrade <[email protected]>
Date:	 2016-09-01 23:42 -0400
Branch:	 default
URL: https://hg.pidgin.im/pidgin/main/rev/7d134a4a87b2

Description:

Ensure all Python scripts are Py3k compatible.

diffstat:

 finch/plugins/pietray.py                     |   2 ++
 libpurple/dbus-analyze-functions.py          |   8 ++++----
 libpurple/dbus-analyze-signals.py            |   2 +-
 libpurple/dbus-analyze-types.py              |   2 +-
 libpurple/plugins/dbus-buddyicons-example.py |   3 ++-
 libpurple/plugins/startup.py                 |   3 ++-
 libpurple/purple-notifications-example       |   7 +++++--
 libpurple/purple-remote                      |  21 +++++++++++++--------
 libpurple/purple-url-handler                 |  10 ++++++----
 9 files changed, 36 insertions(+), 22 deletions(-)

diffs (238 lines):

diff --git a/finch/plugins/pietray.py b/finch/plugins/pietray.py
--- a/finch/plugins/pietray.py
+++ b/finch/plugins/pietray.py
@@ -17,6 +17,8 @@
 # 
 # Sadrul <[email protected]>
 
+from __future__ import absolute_import, division, print_function
+
 import pygtk
 pygtk.require("2.0")
 import gtk
diff --git a/libpurple/dbus-analyze-functions.py b/libpurple/dbus-analyze-functions.py
--- a/libpurple/dbus-analyze-functions.py
+++ b/libpurple/dbus-analyze-functions.py
@@ -1,4 +1,4 @@
-from __future__ import print_function
+from __future__ import absolute_import, division, print_function
 
 import argparse
 import fileinput
@@ -110,7 +110,7 @@ def ctopascal(name):
         newname += word.capitalize()
     return newname
 
-class Parameter:
+class Parameter(object):
     def __init__(self, type, name):
         self.name = name
         self.type = type
@@ -128,7 +128,7 @@ class Parameter:
                     
     fromtokens = staticmethod(fromtokens)
 
-class Binding:
+class Binding(object):
     def __init__(self, functiontext, paramtexts, output=None):
         self.function = Parameter.fromtokens(functiontext.split())
 
@@ -549,7 +549,7 @@ class ServerBinding (Binding):
                                % (name, "len"))
         self.addouttype("ay", name)
 
-class BindingSet:
+class BindingSet(object):
     regexp = r"^(\w[^()]*)\(([^()]*)\)\s*;\s*$";
 
     def __init__(self, inputfile, fprefix, output=None):
diff --git a/libpurple/dbus-analyze-signals.py b/libpurple/dbus-analyze-signals.py
--- a/libpurple/dbus-analyze-signals.py
+++ b/libpurple/dbus-analyze-signals.py
@@ -5,7 +5,7 @@
 #   <signal name="Changed">
 #       <arg name="new_value" type="b"/>
 #   </signal>
-from __future__ import print_function
+from __future__ import absolute_import, division, print_function
 
 import argparse
 import fileinput
diff --git a/libpurple/dbus-analyze-types.py b/libpurple/dbus-analyze-types.py
--- a/libpurple/dbus-analyze-types.py
+++ b/libpurple/dbus-analyze-types.py
@@ -14,7 +14,7 @@
 #           NAME3
 #
 
-from __future__ import print_function
+from __future__ import absolute_import, division, print_function
 
 import argparse
 import fileinput
diff --git a/libpurple/plugins/dbus-buddyicons-example.py b/libpurple/plugins/dbus-buddyicons-example.py
--- a/libpurple/plugins/dbus-buddyicons-example.py
+++ b/libpurple/plugins/dbus-buddyicons-example.py
@@ -21,7 +21,8 @@
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02111-1301  USA
 #
 
-from __future__ import print_function
+from __future__ import absolute_import, division, print_function
+
 import dbus
 
 bus = dbus.SessionBus()
diff --git a/libpurple/plugins/startup.py b/libpurple/plugins/startup.py
--- a/libpurple/plugins/startup.py
+++ b/libpurple/plugins/startup.py
@@ -21,7 +21,8 @@
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02111-1301  USA
 #
 
-from __future__ import print_function
+from __future__ import absolute_import, division, print_function
+
 import sys
 import dbus
 import os
diff --git a/libpurple/purple-notifications-example b/libpurple/purple-notifications-example
--- a/libpurple/purple-notifications-example
+++ b/libpurple/purple-notifications-example
@@ -8,6 +8,8 @@
 # Note that all function names are resolved dynamically, no
 # purple-specific library is needed.
 
+from __future__ import absolute_import, division, print_function
+
 import dbus
 import dbus.glib
 import dbus.decorators
@@ -77,8 +79,9 @@ bus.add_signal_receiver(buddysignedon,
                         dbus_interface = "im.pidgin.purple.PurpleInterface",
                         signal_name = "BuddySignedOn")
 
-print "This is a simple purple notification server."
-print "It shows notifications when your buddy signs on or you get an IM message."
+print("This is a simple purple notification server.")
+print("It shows notifications when your buddy signs on or you get an "
+      "IM message.")
 
 loop = gobject.MainLoop()
 loop.run()
diff --git a/libpurple/purple-remote b/libpurple/purple-remote
--- a/libpurple/purple-remote
+++ b/libpurple/purple-remote
@@ -1,9 +1,14 @@
 #!/usr/bin/env python
 
+from __future__ import absolute_import, division, print_function
+
 import codecs
 import dbus
 import re
-import urllib
+try:
+    from urllib.parse import unquote
+except ImportError:
+    from urllib import unquote
 import sys
 
 import xml.dom.minidom 
@@ -21,14 +26,14 @@ except:
 
 purple = dbus.Interface(obj, "im.pidgin.purple.PurpleInterface")
 
-class CheckedObject:
+class CheckedObject(object):
     def __init__(self, obj):
         self.obj = obj
 
     def __getattr__(self, attr):
         return CheckedAttribute(self, attr)
 
-class CheckedAttribute:
+class CheckedAttribute(object):
     def __init__(self, cobj, attr):
         self.cobj = cobj
         self.attr = attr
@@ -41,7 +46,7 @@ class CheckedAttribute:
         return result
             
 def show_help(requested=False):
-    print """This program uses D-Bus to communicate with purple.
+    print("""This program uses D-Bus to communicate with purple.
 
 Usage:
 
@@ -72,7 +77,7 @@ Examples of commands:
 
     PurpleAccountsFindConnected?name=&protocol=jabber
     PurpleAccountsFindConnected(,jabber)
-""" % sys.argv[0]
+""" % (sys.argv[0], ))
     if (requested):
         sys.exit(0)
     else:
@@ -120,7 +125,7 @@ def execute(uri):
     if paramstring is not None:
         for param in paramstring.split("&"):
             key, value = extendlist(param.split("=",1), 2, "")
-            params[key] = urllib.unquote(value)
+            params[key] = unquote(value)
 
     accountname = params.get("account", "")
 
@@ -233,12 +238,12 @@ if len(sys.argv) == 1:
 elif (sys.argv[1] == "--help" or sys.argv[1] == "-h"):
     show_help(True)
 elif (obj == None):
-    print "No existing libpurple instance detected."
+    print("No existing libpurple instance detected.")
     sys.exit(1);
     
 for arg in sys.argv[1:]:
     output = execute(arg)
 
     if (output != None):
-        print output
+        print(output)
 
diff --git a/libpurple/purple-url-handler b/libpurple/purple-url-handler
--- a/libpurple/purple-url-handler
+++ b/libpurple/purple-url-handler
@@ -1,5 +1,7 @@
 #!/usr/bin/env python
-from __future__ import print_function
+
+from __future__ import absolute_import, division, print_function
+
 import dbus
 import re
 import sys
@@ -20,14 +22,14 @@ except dbus.DBusException as e:
         sys.exit(1)
 purple = dbus.Interface(obj, "im.pidgin.purple.PurpleInterface")
 
-class CheckedObject:
+class CheckedObject(object):
     def __init__(self, obj):
         self.obj = obj
 
     def __getattr__(self, attr):
         return CheckedAttribute(self, attr)
 
-class CheckedAttribute:
+class CheckedAttribute(object):
     def __init__(self, cobj, attr):
         self.cobj = cobj
         self.attr = attr
@@ -384,7 +386,7 @@ def main(argv=sys.argv):
         else:
             print("Unknown protocol: %s" % type)
     except dbus.DBusException as e:
-        print("Error: %s" % e.message)
+        print("Error: %s" % (str(e), ))
         sys.exit(1)
 
 if __name__ == "__main__":

_______________________________________________
Commits mailing list
[email protected]
https://pidgin.im/cgi-bin/mailman/listinfo/commits