From: Yann Dirson <ydirson@altern.org>
Date: Thu, 23 Nov 2006 23:17:12 GMT
Subject: [PATCH 4/4] Bomb out when --ack and --sign are both passed to "refresh".
Message-ID: <20061123231708.9769.53781.stgit@gandelf.nowhere.earth>
In-Reply-To: <20061123230721.9769.38403.stgit@gandelf.nowhere.earth>


Old behaviour was silently ignoring --ack, which could be confusing.

Signed-off-by: Yann Dirson <ydirson@altern.org>
---

 stgit/commands/refresh.py |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/stgit/commands/refresh.py b/stgit/commands/refresh.py
index ea0fe6f..aa5ff78 100644
--- a/stgit/commands/refresh.py
+++ b/stgit/commands/refresh.py
@@ -107,6 +107,8 @@ def func(parser, options, args):
 
     if options.sign:
         sign_str = 'Signed-off-by'
+        if options.ack:
+            raise CmdException, '--ack and --sign were both specified'
     elif options.ack:
         sign_str = 'Acked-by'