From: Eric S. Raymond <esr@thyrsus.com>
Date: Sat, 8 Nov 2014 05:37:16 +0000 (-0500)
Subject: NEWS and test suite polishing.
X-Git-Tag: 0.2~31
X-Git-Url: http://thyrsus.com/gitweb/?a=commitdiff_plain;h=f954fa874e70a1c197839b3ec5f94f6a3a2206dc;p=src.git

NEWS and test suite polishing.
---

diff --git a/NEWS b/NEWS
index 565d411..adbf880 100644
--- a/NEWS
+++ b/NEWS
@@ -2,9 +2,10 @@
 
 Repository head:
    There is a public repository on Gitorious
+   All the initially projected commands except branch are impplemented.
    The bug that caused filures with vi has been fixed.
    ".." is a legal range separator as well as "-".
-   More documentation.
+   There is a regression-test suite, and more documentation.
 
 0.1: 2014-11-07
    Initial proof-of-concept release.
diff --git a/src b/src
index d747b25..295758f 100755
--- a/src
+++ b/src
@@ -112,12 +112,17 @@ class RCSRevisionItem:
         except TypeError, ValueError:
             croak("garbled date %s" % self.date)
 
+def master(arg):
+    return os.path.join(repodir, arg + ",v")
+
 class RCSLog:
     "Encapsulate a revision list and some methods on it"
     def __init__(self, name):
         self.name = name
         self.revlist = []
         self.symbols = {}
+        if not os.path.exists(master(self.name)):
+            croak("%s is not registered" % self.name)
         with popen_or_die("cd %s >/dev/null; rlog %s 2>/dev/null; cd ..>/dev/null" % (repodir, name)) as fp:
             self.revlist = []
             self.lockrevs = []
@@ -365,7 +370,7 @@ def add_method(*args):
     for arg in args:
        if not os.path.exists(arg):
            croak("I see no '%s' here." % arg)
-       elif os.path.exists("%s/%s,v" % (repodir, arg)):
+       elif os.path.exists(master(arg)):
            croak("%s is already registered." % arg)
        else:
            do_or_die("rcs -q -kb -i {0} </dev/null && mv {0},v {1}".format(arg, repodir))
@@ -414,7 +419,7 @@ def commit_method(*args):
     for arg in args:
        if not os.path.exists(arg):
            croak("I see no '%s' here." % arg)
-       elif not os.path.exists("%s/%s,v" % (repodir, arg)):
+       elif not os.path.exists(master(arg)):
            croak("%s is not registered." % arg)
        else:
            if not comment:
@@ -533,8 +538,12 @@ def delete_tags(args):
     if revspec is not None:
         croak("can't accept a revision-spec when deleting tags.")
     for arg in args:
-        with lifted(arg):
-            do_or_die("rcs -n{0} {1}".format(tagname, arg))
+        master = RCSLog(arg)
+        if tagname not in master.symbols:
+            croak("%s is not a tag in %s" % (tagname, arg))
+        else:
+            with lifted(arg):
+                do_or_die("rcs -n{0} {1}".format(tagname, arg))
 
 def set_tags(args):
     if not args:
diff --git a/srctest b/srctest
index 2911fee..d98d624 100755
--- a/srctest
+++ b/srctest
@@ -151,6 +151,6 @@ src $TESTOPTS tag >testfile13
 diff testfile12 testfile13
 check "tag list check after deletion"
 
-# to do: tag, fast-export
+# to do: fast-export
 
 # end