Date: Sun, 17 Sep 2000 15:06:00 -0700 (PDT) From: Kris Kennaway <kris@FreeBSD.org> To: "Brian F. Feldman" <green@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/secure/lib/libssl Makefile src/secure/lib/libcrypto Makefile Makefile.inc Message-ID: <Pine.BSF.4.21.0009171504480.95417-100000@freefall.freebsd.org> In-Reply-To: <200009171521.e8HFLK523802@green.dyndns.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 17 Sep 2000, Brian F. Feldman wrote:
> > > > Overhaul of the build-time include file generation. Don't break in evp.h
> > > > if bootstrapping from a system on which the openssl headers are not
> > > > already present.
> > >
> > > I don't get it; what case does this fix? It was simply _evp.h previously
> > > calling the temporary file evp.h made sure it got used, and building
> > > things without NO_IDEA just didn't work for the obvious reasons. That was
> > > fixed, and I don't see how bootstrapping could have broken (or fixed).
> >
> > Because the file was called _evp.h during the build phase if MAKE_IDEA=NO,
> > and the openssl code looks for evp.h - if you have it in
> > /usr/include/openssl then it picks up that one, which has obvious
> > compatability issues when the include file changes (which is how I
> > discovered it - when integrating the openssl 0.9.6 beta1 release in my
> > local tree)
>
> Hm, that does make some sense then; why don't we just fix the include paths
> that are used during building? If this happened for evp.h, it could occur
> for more files later, I suppose. /usr/include really shouldn't be in any
> include paths at all when building world.
Actually what would have happened is that make world would just break. I
fixed it by installing the file as evp.h in the temporary directory in
both (MAKE_IDEA=YES/NO) cases, instead of just the YES case.
Kris
--
In God we Trust -- all others must submit an X.509 certificate.
-- Charles Forsythe <forsythe@alum.mit.edu>
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0009171504480.95417-100000>