Re: sanity.sh patch for 1.11p1

[Derek R. Price]

Simon Liddington wrote:

> On Mon, 18 Jun 2001, Derek R. Price wrote:
>
> > Simon Liddington wrote:
> >
> > > Running on solaris 2.7 I had to change sanity.sh as per patch attached.
> >
> > The change to the version test has already been checked in.
> >
> > Could you please send clips from check.log demonstrating why the other
> > changes are necessary?
>
> When I first ran "make check" I was missing the space before -> in "->
> main loop with CVSROOT=" in test multiroot2-9. I originally thought it
> was a missing \ before . and 'fixed' this as well without realising it
> was just an escape for . . Then after a rebuild when I discovered
> client/server hadn't got built in the missing space reappeared but I
> left the missing \ 'fix' in. So forget the previous patch but ...
>
> I rebuilt with --disable-server --disable-client for configure and the
> space was missing again so you need the attached patch instead. You
> can see this is correct by looking at server.h where CLIENT_SERVER_STR
> gets set to "" if there's no server support.

Actually, this has been fixed too.  Larry got it a month and a half ago:

2001-04-30  Larry Jones  <larry.jones@sdrc.com>

        * sanity.sh (multiroot2-9a): Rename (from multiroot2-9) to avoid
        duplicate names; fix to work without SERVER_SUPPORT defined.
        (Reported by Pavel Roskin <proski@gnu.org>.)

Thanks though!

Derek

--
Derek Price                      CVS Solutions Architect ( http://CVSHome.org )
mailto:dprice@collab.net         CollabNet ( http://collab.net )
--
My name is not Dr. Death.
My name is not Dr. Death.
My name is not Dr. Death...

          - Bart Simpson on chalkboard, _The Simpsons_