Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 17 Aug 2001 08:44:38 -0700
From:      "David O'Brien" <obrien@FreeBSD.ORG>
To:        Bruce Evans <bde@zeta.org.au>
Cc:        Ruslan Ermilov <ru@FreeBSD.ORG>, current@FreeBSD.ORG, Mark Peek <mark@whistle.com>, Warner Losh <imp@harmony.village.org>
Subject:   Re: Cross builds and upgrade path from 4.x are broken in usr.bin/file
Message-ID:  <20010817084437.C5916@dragon.nuxi.com>
In-Reply-To: <20010815120844.C17139-100000@besplex.bde.org>; from bde@zeta.org.au on Wed, Aug 15, 2001 at 12:40:19PM %2B1000
References:  <20010814203045.A60765@sunbay.com> <20010815120844.C17139-100000@besplex.bde.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Aug 15, 2001 at 12:40:19PM +1000, Bruce Evans wrote:
> On Tue, 14 Aug 2001, Ruslan Ermilov wrote:
> 
> > On Tue, Aug 14, 2001 at 08:55:56AM -0700, David O'Brien wrote:
> > > >From a correctness stand point, building the .mgc files at install time
> > > is the correct thing to do... or maybe we should do both -- doing the
> > > [re]creation of the .mgc files at install time in the cross-[arch-]build
> > > case.
> 
> Not both.

Which do you prefer?  The CC_HOST way, the build-tools way, or the at
install time way?

> > +build-tools: mkmagic
> > +
> > +mkmagic: apprentice.c print-hacked.c
> > +	${CC} -DHAVE_CONFIG_H -DCOMPILE_ONLY \
> > +	    -I${.CURDIR} -I${SRCDIR} -o ${.TARGET} ${.ALLSRC}
> 
> This should use CFLAGS if possible, and should use LDFLAGS, something
> like:
> 
> 	${CC} -DCOMPILE_ONLY {CFLAGS} ${LDFLAGS} -o ${.TARGET} ${.ALLSRC}

The include path in CFLAGS is wrong for this:

    -I/FBSD/5.x/usr.bin/file -I/FBSD/5.x/usr.bin/file/../../contrib/file
    -I/usr/obj/FBSD/5.x/i386/usr/include

namely pointing into /usr/obj.

-- 
-- David  (obrien@FreeBSD.org)

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010817084437.C5916>