#40580: New port: ape -------------------------+-------------------------------- Reporter: fnog@… | Owner: macports-tickets@… Type: submission | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.0 Resolution: | Keywords: Port: ape | -------------------------+-------------------------------- Comment (by ryandesign@…): Thanks. Some observations: * The "`# $Id$`" line is missing. It should be the second line of the portfile, right after the modeline. * The port provides gcc variants which set `configure.compiler`, but the port also includes the line "`configure.args --with-libxc- prefix=${prefix} FCCPP="${configure.cc} -E -ansi"`". This line will be executed ''before'' the variants are evaluated, which means `${configure.cc}` will from that moment on be set to its default value (e.g. /usr/bin/clang on modern Xcode versions). To allow `configure.compiler` to affect `configure.cc`, ensure that you do not access the `configure.cc` variable until after the variants are evaluated. That means enclosing this line inside a `pre-configure` block. * The port provides gcc46, gcc47 and gcc48 variants, but the pre-fetch section checks for not only those but also gcc42, gcc43, gcc44, gcc45 and g95. The port should only check for variants that are defined, or unexpected problems can arise (e.g. if the user requests one of those non- defined variants). * The pre-fetch phase's `ui_msg` line should be removed; it seems like debugging information that users don't need to see. * The pre-fetch block errors out if dependency libxc is not installed with the required variant. Couldn't this be done more simply using the `require_active_variants` procedure? -- Ticket URL: <https://trac.macports.org/ticket/40580#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X