[118041] trunk/dports/databases/mysql56

[Ryan Schmidt]

On Mar 19, 2014, at 19:28, pixilla at macports.org wrote:

> Revision
> 118041
> Author
> pixilla at macports.org
> Date
> 2014-03-19 17:28:20 -0700 (Wed, 19 Mar 2014)
> Log Message
> 
> databases/mysql56:
> - Fix universal build. Closes #42938
> - Use the right compiler. Closes #42943

> Modified: trunk/dports/databases/mysql56/Portfile (118040 => 118041)


> +    # Don't allow mysql to set the compiler to g++
> +    # See http://bazaar.launchpad.net/~mysql/mysql-server/5.6/revision/4223.1.4
> +    # See also #42943
> +    patchfiles-append   patch-CMakeLists.txt.diff
> +    if { (![variant_isset universal] && ${build_arch} eq "i386") || ([variant_isset universal] && [lsearch ${universal_archs} i386] != -1) } {
> +        # Disallow clang versions older than the clang version which caused the problem
> +        #compiler.blacklist {clang < 425}
> +        
> +        # Disallow all clang versions
> +        #compiler.blacklist *clang*
> +        
> +        # switch to /usr/bin/gcc and /usr/bin/g++
> +        # closest to SET(CMAKE_CXX_COMPILER g++) removed in the patchfile
> +        configure.compiler gcc
> +    }

“gcc” could be anything — from gcc 4.0 or 4.2 to llvm-gcc to clang, depending on the Xcode version. Setting configure.compiler to “gcc” is the exact opposite of using the right compiler. MacPorts shouldn’t even have that as a value for configure.compiler.

What is the real problem that you’re trying to solve here?