Closes #14768: os.path.expanduser('~/a') doesn't works correctly when HOME is '/'

This commit is contained in:
Jesus Cea 2012-05-10 05:01:11 +02:00
parent 95f42a86dd
commit f2011e3e49
4 changed files with 6 additions and 2 deletions

View file

@ -267,8 +267,8 @@ def expanduser(path):
except KeyError: except KeyError:
return path return path
userhome = pwent.pw_dir userhome = pwent.pw_dir
userhome = userhome.rstrip('/') or userhome userhome = userhome.rstrip('/')
return userhome + path[i:] return (userhome + path[i:]) or '/'
# Expand paths containing shell variable substitutions. # Expand paths containing shell variable substitutions.

View file

@ -201,6 +201,7 @@ class PosixPathTest(unittest.TestCase):
with test_support.EnvironmentVarGuard() as env: with test_support.EnvironmentVarGuard() as env:
env['HOME'] = '/' env['HOME'] = '/'
self.assertEqual(posixpath.expanduser("~"), "/") self.assertEqual(posixpath.expanduser("~"), "/")
self.assertEqual(posixpath.expanduser("~/foo"), "/foo")
def test_normpath(self): def test_normpath(self):
self.assertEqual(posixpath.normpath(""), ".") self.assertEqual(posixpath.normpath(""), ".")

View file

@ -274,6 +274,7 @@ Doug Fort
John Fouhy John Fouhy
Martin Franklin Martin Franklin
Robin Friedrich Robin Friedrich
Bradley Froehle
Ivan Frohne Ivan Frohne
Jim Fulton Jim Fulton
Tadayoshi Funaba Tadayoshi Funaba

View file

@ -60,6 +60,8 @@ Core and Builtins
Library Library
------- -------
- Issue #14768: os.path.expanduser('~/a') doesn't works correctly when HOME is '/'.
- Issue #13183: Fix pdb skipping frames after hitting a breakpoint and running - Issue #13183: Fix pdb skipping frames after hitting a breakpoint and running
step. Patch by Xavier de Gaye. step. Patch by Xavier de Gaye.