summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArfrever <arfrever@gentoo.org>2011-12-30 11:49:57 -0500
committerAnthony G. Basile <blueness@gentoo.org>2011-12-30 11:50:11 -0500
commitab63707059e517154e3cd5b0f704384e8c74cea0 (patch)
tree3e7a277739c650d94a3d299e0052828d28030ea3 /WebappConfig
parentDon't set unused exception variables. (diff)
downloadwebapp-config-ab63707059e517154e3cd5b0f704384e8c74cea0.tar.gz
webapp-config-ab63707059e517154e3cd5b0f704384e8c74cea0.tar.bz2
webapp-config-ab63707059e517154e3cd5b0f704384e8c74cea0.zip
Update syntax of setting exceptions for compatibility with Python 3.
Reported-By: Arfrever <arfrever@gentoo.org> Signed-off-by: Anthony G. Basile <blueness@gentoo.org>
Diffstat (limited to 'WebappConfig')
-rw-r--r--WebappConfig/config.py4
-rw-r--r--WebappConfig/content.py2
-rw-r--r--WebappConfig/dotconfig.py2
-rw-r--r--WebappConfig/permissions.py8
-rw-r--r--WebappConfig/sandbox.py2
-rw-r--r--WebappConfig/worker.py6
-rw-r--r--WebappConfig/wrapper.py8
7 files changed, 16 insertions, 16 deletions
diff --git a/WebappConfig/config.py b/WebappConfig/config.py
index 83406e8..4e176da 100644
--- a/WebappConfig/config.py
+++ b/WebappConfig/config.py
@@ -54,7 +54,7 @@ class BashConfigParser(ConfigParser.SafeConfigParser):
def get(self, section, option):
try:
return ConfigParser.SafeConfigParser.get(self, section, option)
- except Exception, e:
+ except Exception as e:
error = '\nThere is a problem with your configuration file or' \
' an environment variable.\n' \
'webapp-config tried to read the variable "' + str(option) \
@@ -760,7 +760,7 @@ class Config:
try:
self.config.read(self.__d['my_etcconfig'])
- except Exception, e:
+ except Exception as e:
OUT.die('The config file '
+ self.config.get('USER', 'my_etcconfig') +
' cannot be read by the configuration parser.'
diff --git a/WebappConfig/content.py b/WebappConfig/content.py
index a469921..1683387 100644
--- a/WebappConfig/content.py
+++ b/WebappConfig/content.py
@@ -304,7 +304,7 @@ class Contents:
os.write(fd, '\n'.join(values))
os.close(fd)
- except Exception, e:
+ except Exception as e:
OUT.warn('Failed to write content file ' + dbpath + '!\n'
+ 'Error was: ' + str(e))
else:
diff --git a/WebappConfig/dotconfig.py b/WebappConfig/dotconfig.py
index 4fc47db..be01126 100644
--- a/WebappConfig/dotconfig.py
+++ b/WebappConfig/dotconfig.py
@@ -260,7 +260,7 @@ class DotConfig:
os.write(fd, '\n'.join(info))
os.close(fd)
- except Exception, e:
+ except Exception as e:
OUT.die('Unable to write to ' + self.__dot_config()
+ '\nError was: ' + str(e))
diff --git a/WebappConfig/permissions.py b/WebappConfig/permissions.py
index 64d5f3d..966c610 100644
--- a/WebappConfig/permissions.py
+++ b/WebappConfig/permissions.py
@@ -217,14 +217,14 @@ def get_group(group):
try:
# Try to match the integer to a group id
gid = grp.getgrgid(ngroup)[2]
- except KeyError, e:
+ except KeyError as e:
pass
if gid == -1:
# No success yet. Try to match to the group name
try:
gid = grp.getgrnam(str(group))[2]
- except KeyError, e:
+ except KeyError as e:
raise KeyError('The given group "' + str(group)
+ '" does not exist!')
@@ -265,14 +265,14 @@ def get_user(user):
try:
# Try to match the integer to a user id
uid = pwd.getpwuid(nuser)[2]
- except KeyError, e:
+ except KeyError as e:
pass
if uid == -1:
# No success yet. Try to match to the user name
try:
uid = pwd.getpwnam(str(user))[2]
- except KeyError, e:
+ except KeyError as e:
raise KeyError('The given user "' + str(user)
+ '" does not exist!')
return uid
diff --git a/WebappConfig/sandbox.py b/WebappConfig/sandbox.py
index a15dd61..10873fa 100644
--- a/WebappConfig/sandbox.py
+++ b/WebappConfig/sandbox.py
@@ -96,7 +96,7 @@ class Sandbox:
if not pid:
try:
self._exec(command, self.env, fd_pipes)
- except Exception, e:
+ except Exception as e:
# We need to catch _any_ exception so that it doesn't
# propagate out of this function and cause exiting
# with anything other than os._exit()
diff --git a/WebappConfig/worker.py b/WebappConfig/worker.py
index a5ee172..0841f4a 100644
--- a/WebappConfig/worker.py
+++ b/WebappConfig/worker.py
@@ -500,7 +500,7 @@ class WebappAdd:
my_contenttype = 'sym'
- except Exception, e:
+ except Exception as e:
if self.__v:
OUT.warn('Failed to softlink (' + str(e) + ')')
@@ -515,7 +515,7 @@ class WebappAdd:
my_contenttype = 'sym'
- except Exception, e:
+ except Exception as e:
if self.__v:
OUT.warn('Failed copy symlink (' + str(e) + ')')
@@ -530,7 +530,7 @@ class WebappAdd:
my_contenttype = 'file'
- except Exception, e:
+ except Exception as e:
if self.__v:
OUT.warn('Failed to hardlink (' + str(e) + ')')
diff --git a/WebappConfig/wrapper.py b/WebappConfig/wrapper.py
index 344d809..91185ff 100644
--- a/WebappConfig/wrapper.py
+++ b/WebappConfig/wrapper.py
@@ -48,7 +48,7 @@ def config_protect(cat, pn, pvr, pm):
if pm == "portage":
try:
import portage
- except ImportError, e:
+ except ImportError as e:
OUT.die("Portage libraries not found, quitting:\n%s" % e)
return portage.settings['CONFIG_PROTECT']
@@ -90,7 +90,7 @@ def get_root(config):
if config.config.get('USER', 'package_manager') == "portage":
try:
import portage
- except ImportError, e:
+ except ImportError as e:
OUT.die("Portage libraries not found, quitting:\n%s" % e)
return portage.root
@@ -128,13 +128,13 @@ def package_installed(full_name, pm):
if pm == "portage":
try:
import portage
- except ImportError, e:
+ except ImportError as e:
OUT.die("Portage libraries not found, quitting:\n%s" % e)
try:
t = portage.db[portage.root]["vartree"].dbapi.match(full_name)
# catch the "ambiguous package" Exception
- except ValueError, e:
+ except ValueError as e:
if type(e[0]) == types.ListType:
t = []
for cp in e[0]: