[email protected] writes: > (a) There's already a configure.mtune option. You should use that > ("configure.mtune native") instead of appending to all the flags options.
Oh, awesome, thanks for the tip. > (b) Don't you have to disable binary archives if you set "-mtune=native"? Probably so. Luckily, there are no ports using this port group so this kind of feedback is appreciated. > vq > Sent from my iPhone > >> On Jan 23, 2014, at 3:07 PM, [email protected] wrote: >> >> Revision >> 116291 >> Author >> [email protected] >> Date >> 2014-01-23 12:07:41 -0800 (Thu, 23 Jan 2014) >> Log Message >> >> debug-1.0: add port group for unifying having a debug variant >> Added Paths >> >> trunk/dports/_resources/port1.0/group/debug-1.0.tcl >> Diff >> >> Added: trunk/dports/_resources/port1.0/group/debug-1.0.tcl (0 => 116291) >> >> >> --- trunk/dports/_resources/port1.0/group/debug-1.0.tcl >> (rev 0) >> +++ trunk/dports/_resources/port1.0/group/debug-1.0.tcl 2014-01-23 >> 20:07:41 UTC (rev 116291) >> @@ -0,0 +1,76 @@ >> +# -*- coding: utf-8; mode: tcl; tab-width: 4; indent-tabs-mode: nil; >> c-basic-offset: 4 -*- vim:fenc=utf-8:ft=tcl:et:sw=4:ts=4:sts=4 >> +# $Id$ >> +# >> +# Copyright (c) 2012 The MacPorts Project >> +# All rights reserved. >> +# >> +# Redistribution and use in source and binary forms, with or without >> +# modification, are permitted provided that the following conditions are >> +# met: >> +# >> +# 1. Redistributions of source code must retain the above copyright >> +# notice, this list of conditions and the following disclaimer. >> +# 2. Redistributions in binary form must reproduce the above copyright >> +# notice, this list of conditions and the following disclaimer in the >> +# documentation and/or other materials provided with the distribution. >> +# 3. Neither the name of The MacPorts Project nor the names of its >> +# contributors may be used to endorse or promote products derived from >> +# this software without specific prior written permission. >> +# >> +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS >> +# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT >> +# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR >> +# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT >> +# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, >> +# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT >> +# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, >> +# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY >> +# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT >> +# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE >> +# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. >> +# >> +# >> +# This PortGroup sets up default variants for projects that want m >> +# >> +# Usage: >> +# >> +# PortGroup debug 1.0 >> + >> +configure.cflags-append -mtune=native >> +configure.cxxflags-append -mtune=native >> +configure.fflags-append -mtune=native >> +configure.f90flags-append -mtune=native >> +configure.fcflags-append -mtune=native >> +configure.cppflags-append -mtune=native >> + >> +ui_debug "adding the default debug variant" >> +variant debug description {Enable debug flags and symbols} { >> + configure.cflags-delete -O1 -O2 -O3 -mtune=native -DNDEBUG >> -DNDEBUG=1 >> + configure.cxxflags-delete -O1 -O2 -O3 -mtune=native -DNDEBUG >> -DNDEBUG=1 >> + configure.cppflags-delete -O1 -O2 -O3 -mtune=native -DNDEBUG >> -DNDEBUG=1 >> + configure.fflags-delete -O1 -O2 -O3 -mtune=native -DNDEBUG >> -DNDEBUG=1 >> + configure.f90flags-delete -O1 -O2 -O3 -mtune=native -DNDEBUG >> -DNDEBUG=1 >> + configure.fcflags-delete -O1 -O2 -O3 -mtune=native -DNDEBUG >> -DNDEBUG=1 >> + >> + # certain options break the way cmake configures, so test for cmake >> variable >> + if {[info exists cmake_share_module_dir]} { >> + configure.args-delete -DCMAKE_BUILD_TYPE=Release >> + configure.args-append -DCMAKE_BUILD_TYPE=debugFull >> + } else { >> + configure.args-delete --disable-debug >> + configure.args-append --enable-debug >> + } >> + >> + configure.cflags-append -g -O0 >> + configure.cxxflags-append -g -O0 >> + configure.fflags-append -g -O0 >> + configure.f90flags-append -g -O0 >> + configure.fcflags-append -g -O0 >> +} >> + >> +post-destroot { >> + if {[variant_isset debug]} { >> + ui_debug "Generating the .dSYM bundles because of +debug: find >> ${destroot}${prefix} -type f '(' -name '*.dylib' -or -name '*.so' ')' -exec >> dsymutil {} +" >> + system -W ${destroot}${prefix} "find . -type f '(' -name '*.dylib' >> -or -name '*.so' ')' -exec dsymutil {} +" >> + } >> +} >> _______________________________________________ >> macports-changes mailing list >> [email protected] >> https://lists.macosforge.org/mailman/listinfo/macports-changes _______________________________________________ macports-dev mailing list [email protected] https://lists.macosforge.org/mailman/listinfo/macports-dev
