From d63ae9987624400765650b47bd938a1e554e3a35 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?G=C3=B6k=C3=A7en=20Eraslan?= Date: Tue, 20 Sep 2011 13:48:22 +0000 Subject: [PATCH] actionsapi: Do not silently ignore unlinkDir(file) or unlink(directory) calls. --- pisi/actionsapi/shelltools.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pisi/actionsapi/shelltools.py b/pisi/actionsapi/shelltools.py index f21f1501..3b9f11e2 100644 --- a/pisi/actionsapi/shelltools.py +++ b/pisi/actionsapi/shelltools.py @@ -104,7 +104,7 @@ def unlink(pattern): except OSError: ctx.ui.error(_('ActionsAPI [unlink]: Permission denied: %s.') % (filePath)) elif isDirectory(filePath): - pass + ctx.ui.warning(_('ActionsAPI [unlink]: %s is not a file, use \'unlinkDir\' or \'removeDir\' to remove directories.') % filePath) else: ctx.ui.error(_('ActionsAPI [unlink]: File %s doesn\'t exists.') % (filePath)) @@ -116,7 +116,7 @@ def unlinkDir(sourceDirectory): except OSError: error(_('ActionsAPI [unlinkDir]: Operation not permitted: %s') % (sourceDirectory)) elif isFile(sourceDirectory): - pass + ctx.ui.warning(_('ActionsAPI [unlinkDir]: %s is not a directory, use \'unlink\' or \'remove\' to remove files.') % sourceDirectory) else: error(_('ActionsAPI [unlinkDir]: Directory %s doesn\'t exists.') % (sourceDirectory))