Merge "should use os.path.lexist instead of os.path.exist"

This commit is contained in:
Shawn Pearce 2013-01-28 16:25:54 +00:00 committed by Gerrit Code Review
commit 4baf87f92c

View File

@ -126,11 +126,11 @@ class XmlManifest(object):
self.Override(name) self.Override(name)
try: try:
if os.path.exists(self.manifestFile): if os.path.lexists(self.manifestFile):
os.remove(self.manifestFile) os.remove(self.manifestFile)
os.symlink('manifests/%s' % name, self.manifestFile) os.symlink('manifests/%s' % name, self.manifestFile)
except OSError: except OSError as e:
raise ManifestParseError('cannot link manifest %s' % name) raise ManifestParseError('cannot link manifest %s: %s' % (name, str(e)))
def _RemoteToXml(self, r, doc, root): def _RemoteToXml(self, r, doc, root):
e = doc.createElement('remote') e = doc.createElement('remote')