diff options
author | Ray Lai <ray@cvs.openbsd.org> | 2006-04-17 12:03:20 +0000 |
---|---|---|
committer | Ray Lai <ray@cvs.openbsd.org> | 2006-04-17 12:03:20 +0000 |
commit | 92fe60d3b357050bf6f284775eca285674b0322d (patch) | |
tree | 3ce524b30bb55a73496e8fccfad51a7402bf1943 /regress/usr.bin/rcs | |
parent | 0e03d3dfb6a2be983878e46762b62fe6226a2a85 (diff) |
Relieve checkin_update and checkin_init of the responsibility of
calling rcs_close() so we don't try to rcs_close() the same file
twice if the working file does not exist, resulting in a core dump.
The current code also returns the status of the last file, so if
we do:
$ touch file
$ ci nonexistent file
that will return 0. GNU returns 1. Fix that.
Additionally, it returns -1 on error, which turns into 255. It
should return 1.
OK niallo@
Diffstat (limited to 'regress/usr.bin/rcs')
-rw-r--r-- | regress/usr.bin/rcs/Makefile | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/regress/usr.bin/rcs/Makefile b/regress/usr.bin/rcs/Makefile index f075f098c3f..aedbecde661 100644 --- a/regress/usr.bin/rcs/Makefile +++ b/regress/usr.bin/rcs/Makefile @@ -1,4 +1,4 @@ -# $OpenBSD: Makefile,v 1.19 2006/04/14 15:01:23 ray Exp $ +# $OpenBSD: Makefile,v 1.20 2006/04/17 12:03:19 ray Exp $ # Regression tests by Niall O'Higgins <niallo@openbsd.org>. # ksh -> Makefile by Ray Lai <ray@cyth.net>. @@ -60,6 +60,7 @@ LTESTS= ci-initial \ ci-filemodes \ rcs-iflag \ rlog-lflag \ + ci-nofile \ .for t in ${LTESTS} REGRESS_TARGETS+=test-${t} @@ -481,6 +482,9 @@ test-rlog-lflag: clean @${RLOG} -l file | fgrep -q 'revision 1.2' || \ case "$$?" in 1) exit 0;; esac && exit 1 +test-ci-nofile: + @${CI} -q nonexistent || case "$$?" in 1) exit 0;; esac && exit 1 + clean: @rm -rf ${CLEANFILES} |