[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: sanity.sh patch for 1.11p1
From: |
Derek R. Price |
Subject: |
Re: sanity.sh patch for 1.11p1 |
Date: |
Mon, 18 Jun 2001 11:03:49 -0400 |
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_