Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 29 Jun 2016 01:50:31 +0000 (UTC)
From:      Bryan Drewery <bdrewery@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r302257 - head
Message-ID:  <201606290150.u5T1oVMe062109@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: bdrewery
Date: Wed Jun 29 01:50:30 2016
New Revision: 302257
URL: https://svnweb.freebsd.org/changeset/base/302257

Log:
  WITH_SYSTEM_COMPILER: Avoid using GCC4.8+ logic on native GCC4.2 platforms.
  
  As the XXX notes, these should really be checking MK_GNUCXX since there is
  already a version check in share/mk/src.opts.mk to disable it.  Fixing that
  here is more complex though.  This could also be using X_COMPILER_FEATURES
  but uses X_COMPILER_VERSION to keep in sync with the src.opts.mk logic.
  
  Tested by:	andreast
  Sponsored by:	EMC / Isilon Storage Division
  Approved by:	re (gjb)

Modified:
  head/Makefile.inc1
  head/Makefile.libcompat

Modified: head/Makefile.inc1
==============================================================================
--- head/Makefile.inc1	Tue Jun 28 20:10:30 2016	(r302256)
+++ head/Makefile.inc1	Wed Jun 29 01:50:30 2016	(r302257)
@@ -549,8 +549,10 @@ TARGET_ABI=	gnueabi
 XCFLAGS+=	-isystem ${WORLDTMP}/usr/include -L${WORLDTMP}/usr/lib
 # Force using libc++ for external GCC.
 # XXX: This should be checking MK_GNUCXX == no
+.if ${X_COMPILER_VERSION} >= 40800
 XCXXFLAGS+=	-isystem ${WORLDTMP}/usr/include/c++/v1 -std=c++11 \
 		-nostdinc++ -L${WORLDTMP}/../lib/libc++
+.endif
 .else
 TARGET_ABI?=	unknown
 TARGET_TRIPLE?=	${TARGET_ARCH:C/amd64/x86_64/}-${TARGET_ABI}-freebsd11.0

Modified: head/Makefile.libcompat
==============================================================================
--- head/Makefile.libcompat	Tue Jun 28 20:10:30 2016	(r302256)
+++ head/Makefile.libcompat	Wed Jun 29 01:50:30 2016	(r302257)
@@ -81,8 +81,8 @@ LIBCOMPATCFLAGS+=	-B${LIBCOMPATTMP}/usr/
 LIBCOMPATCFLAGS+=	-isystem ${LIBCOMPATTMP}/usr/include
 # Force using libc++ for external GCC.
 # XXX: This should be checking MK_GNUCXX == no
-.if ${MK_CROSS_COMPILER} == "no" || \
-    (${MK_CLANG_BOOTSTRAP} == "no" && ${MK_GCC_BOOTSTRAP} == "no")
+.if ${X_COMPILER_VERSION} >= 40800 && (${MK_CROSS_COMPILER} == "no" || \
+    (${MK_CLANG_BOOTSTRAP} == "no" && ${MK_GCC_BOOTSTRAP} == "no"))
 LIBCOMPATCXXFLAGS+=	-isystem ${LIBCOMPATTMP}/usr/include/c++/v1 -std=c++11 \
 			-nostdinc++ -L${LIBCOMPAT_OBJTREE}${.CURDIR}/lib/libc++
 .endif



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