From a84e02917af598b0cc050694809dd351913ddd73 Mon Sep 17 00:00:00 2001
From: Jesse Keating <jkeating@redhat.com>
Date: Mon, 20 Sep 2010 13:53:50 -0700
Subject: Fix up tag commands

Minor clean ups from jochen's submission
---
 src/pyfedpkg/__init__.py | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

(limited to 'src/pyfedpkg/__init__.py')

diff --git a/src/pyfedpkg/__init__.py b/src/pyfedpkg/__init__.py
index 6b9b6a1..9e15057 100644
--- a/src/pyfedpkg/__init__.py
+++ b/src/pyfedpkg/__init__.py
@@ -506,16 +506,14 @@ def commit(path=None, message=None, file=None, files=[]):
     _run_command(cmd, cwd=path)
     return
 
-def delete_tag(tagname=None):
-    """Delete a git tag from the repository"""
+def delete_tag(tagname, path=None):
+    """Delete a git tag from the repository found at optional path"""
 
-    if not tagname:
-        raise fedpkgError('Please specified a tagname')
-    cmd = ['git', 'tag']
-    cmd.extend(['-d'])
-    cmd.extend(tagname)
-    _run_command(cmd)
-    log.info ('Tag %s was deleted' % ', '.join(delete))
+    if not path:
+        path = os.getcwd()
+    cmd = ['git', 'tag', '-d', tagname]
+    _run_command(cmd, cwd=path)
+    log.info ('Tag %s was deleted' % tagname)
 
 def diff(path, cached=False, files=[]):
     """Excute a git diff
@@ -657,7 +655,7 @@ def list_tag(tagname=None):
 
     cmd = ['git', 'tag']
     cmd.extend(['-l'])
-    if not tagname == '*':
+    if tagname != '*':
         cmd.extend([tagname])
     # make it so
     _run_command(cmd)
-- 
cgit