On 2013-02-18 07:08, Ryan Schmidt wrote: > > On Feb 16, 2013, at 16:26, [email protected] wrote: > >> Revision: 103165 >> https://trac.macports.org/changeset/103165 >> Author: [email protected] >> 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 _______________________________________________ macports-dev mailing list [email protected] https://lists.macosforge.org/mailman/listinfo/macports-dev
