Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 28 Apr 2015 19:04:45 +0100
From:      Andrew Turner <andrew@fubar.geek.nz>
To:        Garrett Cooper <yaneurabeya@gmail.com>
Cc:        Ed Maste <emaste@FreeBSD.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r282156 - head
Message-ID:  <20150428190445.798316bb@bender>
In-Reply-To: <42F6A579-082C-4159-8E16-1C8B69B0A3AE@gmail.com>
References:  <201504281713.t3SHD5UW096783@svn.freebsd.org> <42F6A579-082C-4159-8E16-1C8B69B0A3AE@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 28 Apr 2015 10:53:57 -0700
Garrett Cooper <yaneurabeya@gmail.com> wrote:

> 
> > On Apr 28, 2015, at 10:13, Ed Maste <emaste@FreeBSD.org> wrote:
> > 
> > Author: emaste
> > Date: Tue Apr 28 17:13:05 2015
> > New Revision: 282156
> > URL: https://svnweb.freebsd.org/changeset/base/282156
> > 
> > Log:
> >  Don't add arm64 to universe builds if the user provided a TARGETS
> > list
> > 
> >  Differential Revision:    https://reviews.freebsd.org/D2375
> >  Reported by:    andrew
> >  Reviewed by:    andrew, imp
> >  Sponsored by:    The FreeBSD Foundation
> > 
> > Modified:
> >  head/Makefile
> > 
> > Modified: head/Makefile
> > ==============================================================================
> > --- head/Makefile    Tue Apr 28 17:05:55 2015    (r282155)
> > +++ head/Makefile    Tue Apr 28 17:13:05 2015    (r282156)
> > @@ -373,19 +373,19 @@ kernel-toolchains:
> > # existing system is.
> > #
> > .if make(universe) || make(universe_kernels) || make(tinderbox) ||
> > make(targets) -TARGETS?=amd64 arm i386 mips pc98 powerpc sparc64
> > # XXX Add arm64 to universe only if we have an external binutils
> > installed. # It does not build with the in-tree linker.
> > .if exists(/usr/local/aarch64-freebsd/bin/ld)
> > -TARGETS+=arm64
> > -TARGET_ARCHES_arm64?=    aarch64
> > -.else
> > +UNIVERSE_arm64=arm64
> > +.elif empty(${TARGETS})
> > universe: universe_arm64_skip
> > universe_epilogue: universe_arm64_skip
> > universe_arm64_skip: universe_prologue
> >    @echo ">> arm64 skipped - install aarch64-binutils port or
> > package to build" .endif
> > +TARGETS?=amd64 arm
> 
> Why just amd64/arm here?

You're missing part of the line, it contains all architectures with
arm64 being optional:

+TARGETS?=amd64 arm ${UNIVERSE_arm64} i386 mips pc98 powerpc sparc64

Andrew



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