From owner-svn-ports-head@freebsd.org Sun Jun 19 12:07:00 2016 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id CBC17A7825C; Sun, 19 Jun 2016 12:07:00 +0000 (UTC) (envelope-from gerald@pfeifer.com) Received: from ainaz.pair.com (ainaz.pair.com [209.68.2.66]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 101BD1E91; Sun, 19 Jun 2016 12:07:00 +0000 (UTC) (envelope-from gerald@pfeifer.com) Received: from anthias (vie-91-186-158-235.dsl.sil.at [91.186.158.235]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ainaz.pair.com (Postfix) with ESMTPSA id 750443F427; Sun, 19 Jun 2016 08:06:57 -0400 (EDT) Date: Sun, 19 Jun 2016 14:06:56 +0200 (CEST) From: Gerald Pfeifer To: John Marino cc: Andreas Tobler , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r416339 - head/lang/gcc5-aux/files In-Reply-To: <98f0eac7-3559-81fd-29cd-bd234cec6b6b@marino.st> Message-ID: References: <201606032131.u53LV4K3009774@repo.freebsd.org> <98f0eac7-3559-81fd-29cd-bd234cec6b6b@marino.st> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 19 Jun 2016 12:07:00 -0000 Hi John, On Sat, 4 Jun 2016, John Marino wrote: >> Author: andreast (src committer) >> Date: Fri Jun 3 21:31:04 2016 >> New Revision: 416339 >> URL: https://svnweb.freebsd.org/changeset/ports/416339 >> >> Log: >> Skip armv6hf support and move it into armv6. > These aren't going to work. > I don't use regular patches, at all. They can interfere with the generated > diff-* aggregate patches. Any new patch should be pulled into there. > > But what are these patches really for? Ada doesn't build on ARM due to the > lack of a bootstrap compiler and interest. Should we just set ONLY_FOR_ARCH > because without Ada, people should probably use the stock lang/gcc* ports. I believe what happened is that as Andreas and me discussed fixing armv6hf / armv6 support and I approved changes to "my" lang/gcc* ports, he (or some tab completion) unintentionally also hit this lang/gcc5-aux ports. And, I agree, unless someone is looking for Ada, using the stock lang/gcc* ports probably makes most sense. Gerald PS: By the way, how about using the standard FORTRAN instead of FORT in your ports, per the patch below? Index: Makefile =================================================================== --- Makefile (revision 417113) +++ Makefile (working copy) @@ -30,12 +30,11 @@ LDFLAGS+= -L${LOCALBASE}/lib OPTIONS_GROUP= Standard Bootstrap -OPTIONS_GROUP_Standard= FORT OBJC NLS TESTSUITE ALLSTAGES STATIC +OPTIONS_GROUP_Standard= FORTRAN OBJC NLS TESTSUITE ALLSTAGES STATIC OPTIONS_GROUP_Bootstrap=BOOTSTRAP -OPTIONS_DEFAULT= FORT OBJC +OPTIONS_DEFAULT= FORTRAN OBJC NO_OPTIONS_SORT= yes -FORT_DESC= Fortran language support OBJC_DESC= Objective-C language support TESTSUITE_DESC= Activate test support ALLSTAGES_DESC= Build compiler with itself (twice, brings in libcc1) @@ -113,7 +112,7 @@ ada-bootstrap.x86_64.freebsd.100B.tar.bz2:boot .endif -.if ${PORT_OPTIONS:MFORT} && !${PORT_OPTIONS:MBOOTSTRAP} +.if ${PORT_OPTIONS:MFORTRAN} && !${PORT_OPTIONS:MBOOTSTRAP} LANGS+= fortran EXTRA_CONFIG+= --enable-libquadmath .else @@ -238,7 +237,7 @@ ${MAKE_CMD} -j${MAKE_JOBS_NUMBER} -sk check-c test-fortran: -. if ${PORT_OPTIONS:MFORT} +. if ${PORT_OPTIONS:MFORTRAN} cd ${BUILDDIR} && ${SETENV} ${ADA_MAKE_ENV} \ ${MAKE_CMD} -j${MAKE_JOBS_NUMBER} -sk check-fortran . endif