Will this patch break anything?  I assume it might break running
davautocheck against an installed svn/apxs pair --- i.e., running
basic_tests.py against /usr/local/bin/apxs and /usr/local/bin/svn, out
of a working copy (or tarball) that not even 'configure' had been run in.

Thoughts?

[[[
Followup to r1507889:

* subversion/tests/cmdline/davautocheck.sh
  (APXS): Stop guessing where APXS might be.  configure will have found
    it for us.
]]]

Index: subversion/tests/cmdline/davautocheck.sh
===================================================================
--- subversion/tests/cmdline/davautocheck.sh    (revision 1507891)
+++ subversion/tests/cmdline/davautocheck.sh    (working copy)
@@ -190,10 +190,6 @@ if [ ${APXS:+set} ]; then
 elif APXS=$(grep '^APXS' $ABS_BUILDDIR/Makefile | sed 's/^APXS *= *//') && \
      [ -n "$APXS" ]; then
   :
-elif APXS=$(which apxs); then
-  :
-elif APXS=$(which apxs2); then
-  :
 else
   fail "neither apxs or apxs2 found - required to run davautocheck"
 fi

Reply via email to