[macports-ports] branch master updated: ghc and ghc-bootstrap: specify compiler to clang-4.0

[Ryan Schmidt]

On Sep 26, 2017, at 14:11, Ken wrote:

> Ken (kencu) pushed a commit to branch master
> in repository macports-ports.
> 
> 
> https://github.com/macports/macports-ports/commit/b6060617db664a7ca586df1d8894da2054f7db75
> 
> The following commit(s) were added to refs/heads/master by this push:
> 
>      new b606061  ghc and ghc-bootstrap: specify compiler to clang-4.0
> 
> b606061 is described below
> 
> 
> commit b6060617db664a7ca586df1d8894da2054f7db75
> 
> Author: Ken Cunningham <kencu at macports.org>
> AuthorDate: Sun Sep 24 15:28:31 2017 -0700
> 
>     ghc and ghc-bootstrap: specify compiler to clang-4.0
>     
>     on older systems and on 10.11
>     and add a build and runtime dependency for clang-4.0
>     as this compiler needs to be present when ghc is used.
>     closes: https://trac.macports.org/ticket/54363
>     closes: https://trac.macports.org/ticket/53709
>     closes: https://trac.macports.org/ticket/50962
>     closes: https://trac.macports.org/ticket/50480


> +# ghc needs a build and runtime dependency on the compiler used to build it
> +# same is also set in ghc-bootstrap. clang-4.0 works, is needed on older systems anyway
> +# also /usr/bin/clang does not work on 10.11, so override it there
> +if { ${os.platform} eq "darwin" && ( ${os.major} < 13 || ${os.major} == 15 ) } {
> +    compiler.whitelist    macports-clang-4.0
> +    depends_build-append  port:clang-4.0
> +    depends_run-append    port:clang-4.0
> +}
> +
>  set bootstraproot ${prefix}/share/ghc-bootstrap
>  set llvmPrefix  ${prefix}/libexec/llvm-3.5
>  configure.args  --with-ghc=${bootstraproot}/bin/ghc \

MacPorts automatically adds a build dependency on the compiler it uses, if that compiler comes from MacPorts and if that dependency has not already been added, so having already specified the compiler whitelist you don't then also need to add the build dependency. Though you do of course need the runtime dependency, as you have it, for reasons specific to ghc.

So, now ghc requires both llvm-3.5 and llvm-4.0? Seems excessive. Could it be fixed to need only a single version of llvm?