[macports-ports] 03/06: cxx11 PG: do not force the use of depends_lib-append

Ryan Schmidt ryandesign at macports.org
Tue Dec 5 05:23:26 UTC 2017


On Oct 21, 2017, at 23:10, Marcus Calhoun-Lopez wrote:

> Marcus Calhoun-Lopez (MarcusCalhoun-Lopez) pushed a commit to branch master
> in repository macports-ports.
> 
> 
> https://github.com/macports/macports-ports/commit/9ae6b8e85dc534d9d23144f0664b4d3790f35187
> 
> commit 9ae6b8e85dc534d9d23144f0664b4d3790f35187
> 
> Author: Marcus Calhoun-Lopez
> AuthorDate: Sat Oct 21 21:02:00 2017 -0700
> 
> 
>     cxx11 PG: do not force the use of depends_lib-append
> 
> ---
>  _resources/port1.0/group/cxx11-1.1.tcl | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/_resources/port1.0/group/cxx11-1.1.tcl b/_resources/port1.0/group/cxx11-1.1.tcl
> index df41a74..f9b7534 100644
> --- a/_resources/port1.0/group/cxx11-1.1.tcl
> +++ b/_resources/port1.0/group/cxx11-1.1.tcl
> @@ -44,8 +44,13 @@ if {${cxx_stdlib} eq "libstdc++" } {
>      # see https://trac.macports.org/ticket/53194
>      configure.cxx_stdlib macports-libstdc++
>  
> -    # see https://trac.macports.org/ticket/54766
> -    depends_lib-append port:libgcc
> +    proc register_gcc_dependents {} {
> +        global os.major
> +        # see https://trac.macports.org/ticket/54766
> +        depends_lib-append port:libgcc
> +    }
> +    # do not force all Portfiles to switch from depends_lib to depends_lib-append
> +    port::register_callback register_gcc_dependents

Shouldn't it be "dependencies" not "dependents"?

Also, why `global os.major`? It's not used...

The proc name should probably begin with "cxx11." to avoid possible collisions with other portgroups.

How about just "proc cxx11.add_dependencies"?



More information about the macports-dev mailing list