Skip to content

Commit

Permalink
pylint for ClientUtilities
Browse files Browse the repository at this point in the history
  • Loading branch information
mmascher committed Dec 18, 2015
1 parent 6416ece commit d629737
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/python/CRABClient/ClientUtilities.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ def filter(self, record):
def removecolor(text):
if not text:
return text
for color, colorval in colors.colordict.iteritems():
for dummyColor, colorval in colors.colordict.iteritems():
if colorval in text:
text = text.replace(colorval, '')
return text
Expand Down Expand Up @@ -200,7 +200,7 @@ def uploadlogfile(logger, proxyfilename, logfilename = None, logpath = None, ins
cmd = 'env'
logger.debug('Running env command')
pipe = subprocess.Popen(cmd, stdout = subprocess.PIPE, stderr = subprocess.PIPE, shell = True)
stdout, stderr = pipe.communicate()
stdout, dummyStderr = pipe.communicate()
logger.debug('\n\n\nUSER ENVIROMENT\n%s' % stdout)
except Exception as se:
logger.debug('Failed to get the user env\nException message: %s' % (se))
Expand Down Expand Up @@ -430,8 +430,8 @@ def getWorkArea(projdir):
return requestarea, requestname


def loadCache(dir, logger):
requestarea, requestname = getWorkArea(dir)
def loadCache(mydir, logger):
requestarea, dummyRequestname = getWorkArea(mydir)
cachename = os.path.join(requestarea, '.requestcache')
#Check if the directory exists.
if not os.path.isdir(requestarea):
Expand Down Expand Up @@ -660,7 +660,7 @@ def server_info(subresource, server, proxyfilename, baseurl, **kwargs):
server = CRABClient.Emulator.getEmulator('rest')(server, proxyfilename, proxyfilename, version=__version__)
requestdict = {'subresource': subresource}
requestdict.update(**kwargs)
dictresult, status, reason = server.get(baseurl, requestdict)
dictresult, dummyStatus, dummyReason = server.get(baseurl, requestdict)

return dictresult['result'][0]

Expand Down

0 comments on commit d629737

Please sign in to comment.