[103165] trunk/dports/devel/fossil

[Rainer Müller]

On 2013-02-18 07:08, Ryan Schmidt wrote:
> 
> On Feb 16, 2013, at 16:26, ciserlohn at macports.org wrote:
> 
>> Revision: 103165
>>          https://trac.macports.org/changeset/103165
>> Author:   ciserlohn at macports.org
>> Date:     2013-02-16 14:26:55 -0800 (Sat, 16 Feb 2013)
>> Log Message:
>> -----------
>> fossil: update to version 1.25
>>
>> * add dependency to libiconv and append appropriate ldflag
>> * add patch to remove gcc only option when compiling with clang
>>
>> Modified Paths:
>> --------------
>>    trunk/dports/devel/fossil/Portfile
>>
>> Added Paths:
>> -----------
>>    trunk/dports/devel/fossil/files/
>>    trunk/dports/devel/fossil/files/patch-autosetup-cc.tcl_gstab_option_is_gcc_only.diff
> 
> 
>> +if {${configure.compiler} == "clang"} {
>> +    patchfiles-append   patch-autosetup-cc.tcl_gstab_option_is_gcc_only.diff
>> +}
> 
> You should use "[string match "*clang*" ${configure.compiler}]" to also catch MacPorts clang versions.

It would be okay to remove -gstabs for all compilers as it enables debug
symbols, which we usually do not want when building ports anyway.

Recently, I made the same change for OpenOCD which appears to use the
same build system: https://trac.macports.org/changeset/102718

Rainer