Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 24 Nov 2012 20:08:07 +1100
From:      Dima Panov <fluffy@fluffy.khv.ru>
To:        "Jason E. Hale" <jhale@freebsd.org>
Cc:        svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org, Dima Panov <fluffy@freebsd.org>
Subject:   Re: svn commit: r307701 - head/graphics/vigra
Message-ID:  <50B08E77.80100@fluffy.khv.ru>
In-Reply-To: <23752824.Nt9BvllcTf@mocha.verizon.net>
References:  <201211240225.qAO2PdBV045963@svn.freebsd.org> <23752824.Nt9BvllcTf@mocha.verizon.net>

next in thread | previous in thread | raw e-mail | index | archive | help
24.11.2012 17:17, Jason E. Hale пишет:
> On Saturday, November 24, 2012 02:25:38 Dima Panov wrote:
>> Author: fluffy
>> Date: Sat Nov 24 02:25:38 2012
>> New Revision: 307701
>> URL: http://svnweb.freebsd.org/changeset/ports/307701
>>
>> Log:
>>    - Force to use gcc46+ if CC is not Clang.
>>
>>    Feature safe:	yes
>>
>> Modified:
>>    head/graphics/vigra/Makefile
>>
>> Modified: head/graphics/vigra/Makefile
>> ============================================================================
>> == --- head/graphics/vigra/Makefile	Sat Nov 24 02:23:00 2012	(r307700) +++
>> head/graphics/vigra/Makefile	Sat Nov 24 02:25:38 2012	(r307701) @@ -56,6
>> +56,12 @@ PLIST_SUB+=	PYTHON="@comment "
>>
>>   .include <bsd.port.pre.mk>
>>
>> +# Check for clang, else use gcc46+
>> +CPP_check!=	${CPP} --version | tr -d '()'
>> +.if !${CPP_check:Mclang}
>> +USE_GCC=	4.6+
>> +.endif
>> +
> I think it would be better to avoid the '!=' expansion and check for
> clang/clang++ with something like this:
>
> .if ${CC:T:Mclang} != "clang" && ${CXX:T:Mclang++} != "clang++"
> USE_GCC=        4.6+
> .endif
>
This construction doesn't cover situation when cc = clang (-current, for 
example, where WITH_CLANG_IS_CC is default), so I've used own check..


-- 
Dima Panov (fluffy@FreeBSD.org)
(KDE, Office)@FreeBSD team

Facebook: http://www.facebook.com/fluffy.khv
IRC: fluffy@EFNet, fluffykhv@FreeNode
twitter: fluffy_khv | skype: dima.panov




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