[28575] in Source-Commits
delete commit: Correctly call os.unlink
daemon@ATHENA.MIT.EDU (Jonathan D Reed)
Mon Dec 21 14:32:12 2015
Date: Mon, 21 Dec 2015 14:32:10 -0500
From: Jonathan D Reed <jdreed@mit.edu>
Message-Id: <201512211932.tBLJWAt4012121@drugstore.mit.edu>
To: source-commits@mit.edu
https://github.com/mit-athena/delete/commit/c70dde8ffd56643666dc4e74783dcf83edaa04e7
commit c70dde8ffd56643666dc4e74783dcf83edaa04e7
Author: Jonathan Reed <jdreed@mit.edu>
Date: Mon Dec 21 14:31:26 2015 -0500
Correctly call os.unlink
unlink is in os, not os.path (Trac: #1569)
delete | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/delete b/delete
index deafbe2..054d1cd 100755
--- a/delete
+++ b/delete
@@ -76,7 +76,7 @@ def actually_delete(filename, options):
if os.path.isdir(newname) and not os.path.islink(newname):
shutil.rmtree(newname)
else:
- os.path.unlink(newname)
+ os.unlink(newname)
assert not os.path.exists(newname), "Fatal error: new path exists"
logger.debug("Move %s to %s", filename, newname)
# Maybe we can just use os.rename here