From 5cac46dd413766411dc12758d53b6a7e6b691171 Mon Sep 17 00:00:00 2001 From: Jack Diederich Date: Mon, 22 Feb 2010 21:27:38 +0000 Subject: * fix issue#7476 --- Lib/pipes.py | 3 +++ Lib/test/test_pipes.py | 2 ++ Misc/ACKS | 1 + 3 files changed, 6 insertions(+) diff --git a/Lib/pipes.py b/Lib/pipes.py index 6dcc997..25e9915 100644 --- a/Lib/pipes.py +++ b/Lib/pipes.py @@ -267,10 +267,13 @@ _safechars = string.ascii_letters + string.digits + '!@%_-+=:,./' # Safe unquote _funnychars = '"`$\\' # Unsafe inside "double quotes" def quote(file): + ''' return a shell-escaped version of the file string ''' for c in file: if c not in _safechars: break else: + if not file: + return "''" return file if '\'' not in file: return '\'' + file + '\'' diff --git a/Lib/test/test_pipes.py b/Lib/test/test_pipes.py index d1053e8..a638598 100644 --- a/Lib/test/test_pipes.py +++ b/Lib/test/test_pipes.py @@ -76,6 +76,8 @@ class SimplePipeTests(unittest.TestCase): self.assertEqual(pipes.quote("test%s'name'" % u), '"test\\%s\'name\'"' % u) + self.assertEqual(pipes.quote(''), "''") + def testRepr(self): t = pipes.Template() self.assertEqual(repr(t), "