diff options
author | Kaleb S. KEITHLEY <kkeithle@redhat.com> | 2018-05-30 08:15:29 -0400 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-06-04 19:55:35 +0000 |
commit | 7cdcd9b022180ee279f6408f7daaa882a8266f3a (patch) | |
tree | db92267be85f33d809d44d02b22ba63847999c7a /geo-replication | |
parent | 3894f4262d53d1c1c593a78b21d72ba1103c86cd (diff) |
core/various: python3 compat, prepare for python2 -> python3
see https://review.gluster.org/#/c/19788/,
https://review.gluster.org/#/c/19871/, and
https://review.gluster.org/#/c/19952/
This patch adds version agnostic imports for urllib, cpickle,
socketserver, _thread, queue, etc., suggested by Aravinda in
https://review.gluster.org/#/c/19767/1
Note: Fedora packaging guidelines require explicit shebangs, so
popular practices like #!/usr/bin/env python and #!/usr/bin/python
are not allowed; they must be #!/usr/bin/python2 or #!/usr/bin/python3
Note: Selected small fixes from 2to3 utility. Specifically apply,
basestring, funcattrs, idioms, numliterals, set_literal, types, urllib,
and zip have already been applied.
Note: these 2to3 fixes report no changes are necessary: exec, execfile,
exitfunc, filter, getcwdu, intern, itertools, metaclass, methodattrs, ne,
next, nonzero, operator, paren, raw_input, reduce, reload, renames, repr,
standarderror, sys_exc, throw, tuple_params, xreadlines.
Change-Id: I8d393064a1837874d8b4bc87c8ce05c679664642
updates: #411
Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
Diffstat (limited to 'geo-replication')
-rw-r--r-- | geo-replication/syncdaemon/argsupgrade.py | 10 | ||||
-rw-r--r-- | geo-replication/syncdaemon/gsyncd.py | 11 | ||||
-rw-r--r-- | geo-replication/syncdaemon/gsyncdconfig.py | 16 | ||||
-rw-r--r-- | geo-replication/syncdaemon/gsyncdstatus.py | 5 | ||||
-rw-r--r-- | geo-replication/syncdaemon/logutils.py | 11 | ||||
-rw-r--r-- | geo-replication/syncdaemon/repce.py | 15 | ||||
-rw-r--r-- | geo-replication/syncdaemon/subcmds.py | 10 | ||||
-rw-r--r-- | geo-replication/syncdaemon/syncdutils.py | 5 |
8 files changed, 77 insertions, 6 deletions
diff --git a/geo-replication/syncdaemon/argsupgrade.py b/geo-replication/syncdaemon/argsupgrade.py index a97c748c40b..632271daf81 100644 --- a/geo-replication/syncdaemon/argsupgrade.py +++ b/geo-replication/syncdaemon/argsupgrade.py @@ -1,3 +1,13 @@ +# -*- coding: utf-8 -*- +# +# Copyright (c) 2016 Red Hat, Inc. <http://www.redhat.com> +# This file is part of GlusterFS. +# +# This file is licensed to you under your choice of the GNU Lesser +# General Public License, version 3 or any later version (LGPLv3 or +# later), or the GNU General Public License, version 2 (GPLv2), in all +# cases as published by the Free Software Foundation. +# # Converts old style args into new style args from __future__ import print_function diff --git a/geo-replication/syncdaemon/gsyncd.py b/geo-replication/syncdaemon/gsyncd.py index 1ab65877467..77fca4c9d1f 100644 --- a/geo-replication/syncdaemon/gsyncd.py +++ b/geo-replication/syncdaemon/gsyncd.py @@ -1,4 +1,15 @@ #!/usr/bin/python2 +# -*- coding: utf-8 -*- +# +# Copyright (c) 2016 Red Hat, Inc. <http://www.redhat.com> +# This file is part of GlusterFS. +# +# This file is licensed to you under your choice of the GNU Lesser +# General Public License, version 3 or any later version (LGPLv3 or +# later), or the GNU General Public License, version 2 (GPLv2), in all +# cases as published by the Free Software Foundation. +# + from argparse import ArgumentParser import time import os diff --git a/geo-replication/syncdaemon/gsyncdconfig.py b/geo-replication/syncdaemon/gsyncdconfig.py index 9fb2b9e5323..8403477dac3 100644 --- a/geo-replication/syncdaemon/gsyncdconfig.py +++ b/geo-replication/syncdaemon/gsyncdconfig.py @@ -1,4 +1,18 @@ -from ConfigParser import ConfigParser, NoSectionError +# -*- coding: utf-8 -*- +# +# Copyright (c) 2016 Red Hat, Inc. <http://www.redhat.com> +# This file is part of GlusterFS. +# +# This file is licensed to you under your choice of the GNU Lesser +# General Public License, version 3 or any later version (LGPLv3 or +# later), or the GNU General Public License, version 2 (GPLv2), in all +# cases as published by the Free Software Foundation. +# + +try: + from configparser import ConfigParser, NoSectionError +except ImportError: + from ConfigParser import ConfigParser, NoSectionError import os from string import Template from datetime import datetime diff --git a/geo-replication/syncdaemon/gsyncdstatus.py b/geo-replication/syncdaemon/gsyncdstatus.py index e75f9dcd62c..e8a810f4b38 100644 --- a/geo-replication/syncdaemon/gsyncdstatus.py +++ b/geo-replication/syncdaemon/gsyncdstatus.py @@ -13,7 +13,10 @@ from __future__ import print_function import fcntl import os import tempfile -import urllib +try: + import urllib.parse as urllib +except ImportError: + import urllib import json import time from datetime import datetime diff --git a/geo-replication/syncdaemon/logutils.py b/geo-replication/syncdaemon/logutils.py index f00685cd92c..01ae7852f23 100644 --- a/geo-replication/syncdaemon/logutils.py +++ b/geo-replication/syncdaemon/logutils.py @@ -1,3 +1,14 @@ +# -*- coding: utf-8 -*- +# +# Copyright (c) 2016 Red Hat, Inc. <http://www.redhat.com> +# This file is part of GlusterFS. +# +# This file is licensed to you under your choice of the GNU Lesser +# General Public License, version 3 or any later version (LGPLv3 or +# later), or the GNU General Public License, version 2 (GPLv2), in all +# cases as published by the Free Software Foundation. +# + import logging from logging import Logger, handlers import sys diff --git a/geo-replication/syncdaemon/repce.py b/geo-replication/syncdaemon/repce.py index 1fb90c44032..f819a89bfee 100644 --- a/geo-replication/syncdaemon/repce.py +++ b/geo-replication/syncdaemon/repce.py @@ -13,9 +13,18 @@ import sys import time import logging from threading import Condition -import thread -from Queue import Queue -import cPickle as pickle +try: + import _thread +except ImportError: + import thread as _thread +try: + from queue import Queue +except ImportError: + from Queue import Queue +try: + import cPickle as pickle +except ImportError: + import pickle from syncdutils import Thread, select, lf diff --git a/geo-replication/syncdaemon/subcmds.py b/geo-replication/syncdaemon/subcmds.py index b9e02855392..11d263d7e03 100644 --- a/geo-replication/syncdaemon/subcmds.py +++ b/geo-replication/syncdaemon/subcmds.py @@ -1,3 +1,13 @@ +# -*- coding: utf-8 -*- +# +# Copyright (c) 2016 Red Hat, Inc. <http://www.redhat.com> +# This file is part of GlusterFS. +# +# This file is licensed to you under your choice of the GNU Lesser +# General Public License, version 3 or any later version (LGPLv3 or +# later), or the GNU General Public License, version 2 (GPLv2), in all +# cases as published by the Free Software Foundation. +# from __future__ import print_function from syncdutils import lf diff --git a/geo-replication/syncdaemon/syncdutils.py b/geo-replication/syncdaemon/syncdutils.py index 1e40ff56858..ab6753ee96c 100644 --- a/geo-replication/syncdaemon/syncdutils.py +++ b/geo-replication/syncdaemon/syncdutils.py @@ -28,7 +28,10 @@ import select as oselect from os import waitpid as owaitpid import xml.etree.ElementTree as XET from select import error as SelectError -from cPickle import PickleError +try: + from cPickle import PickleError +except ImportError: + from pickle import PickleError from conf import GLUSTERFS_LIBEXECDIR, UUID_FILE sys.path.insert(1, GLUSTERFS_LIBEXECDIR) |