Hello!
Now everybody can run "sanity.sh -r" - my patch removes any dependency on
the system-wide rsh.
Instead, sanity.sh creates a file named "trsh" - trivial remote shell that
simply ignores the hostname and (if given) the username.
It is very important that many people run sanity.sh now when serious
changes are being made in the CVS code.
Regards,
Pavel Roskin
========================================
Index: src/ChangeLog
--- src/ChangeLog Tue Jun 20 22:07:21 2000
+++ src/ChangeLog Tue Jun 20 23:59:12 2000
@@ -1 +1,6 @@
+2000-06-20 Pavel Roskin <[EMAIL PROTECTED]>
+
+ * sanity.sh (crerepos): Create and use a trivial rsh replacement
+ to avoid dependency on the system rsh.
+
2000-06-19 Larry Jones <[EMAIL PROTECTED]>
Index: src/sanity.sh
--- src/sanity.sh Wed Jun 14 21:07:09 2000
+++ src/sanity.sh Wed Jun 21 00:01:37 2000
@@ -13384,23 +13384,26 @@
# Use :ext: rather than :fork:. Most of the tests use :fork:,
# so we want to make sure that we test :ext: _somewhere_.
- # Maybe a bit dubious in the sense that people need to
- # have rsh working to run the tests, but at least it
- # isn't inetd :-). Might want to think harder about this -
- # maybe try :ext:, and if it fails, print a (single, nice)
- # message and fall back to :fork:. Maybe testing :ext:
- # with our own CVS_RSH rather than worrying about a system one
- # would do the trick.
+ # We don't want to worry about the system configuration
+ # CVS_RSH is set to trsh - trivial remote shell that
+ # just skips the hostname and the username (if given)
+
+ cat >trsh <<'EOF'
+#!/bin/sh
+shift
+if test x"$1" = x-l; then
+ shift
+ shift
+fi
+eval "$@"
+EOF
+ chmod +x trsh
+ CVS_RSH=`pwd`/trsh
+ export CVS_RSH
# Note that we set CVS_SERVER at the beginning.
CREREPOS_ROOT=:ext:`hostname`:${TESTDIR}/crerepos
- # If we're going to do remote testing, make sure 'rsh' works first.
- host="`hostname`"
- if test "x`${CVS_RSH-rsh} $host -n 'echo hi'`" != "xhi"; then
- echo "ERROR: cannot test remote CVS, because \`rsh $host' fails." >&2
- exit 1
- fi
fi
if test "x$remote" = "xno"; then
@@ -13505,6 +13508,7 @@
rm -r 1
rm -rf ${CVSROOT_DIRNAME}/first-dir ${TESTDIR}/crerepos
+ rm -f trsh
;;
rcs)
========================================
