Opened 7 years ago

Closed 7 years ago

#55016 closed defect (fixed)

salt: default_variants issue

Reported by: mf2k (Frank Schima) Owned by: aphor (Jeremy McMillan)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: salt

Description (last modified by mf2k (Frank Schima))

The salt portfile needs to add a check before setting the default variant to +python27. I have +python36 in my variants.conf file and I see the following error: 

$ port info salt
Error: salt: Variant python27 conflicts with python36
Error: Unable to open port: Error evaluating variants

The fix is to change: 

    default_variants    +python27

to 

    if {![variant_isset python34] && ![variant_isset python35] && ![variant_isset python36]} {
        default_variants +python27
    }

I can make this change if the maintainer approves?

Change History (4)

comment:1 Changed 7 years ago by mf2k (Frank Schima)

Description: modified (diff)

comment:2 Changed 7 years ago by mf2k (Frank Schima)

ping ?

comment:3 Changed 7 years ago by mf2k (Frank Schima)

This can be directly seen as follows: 

$ port info salt +python36
Error: salt: Variant python27 conflicts with python36
Error: Unable to open port: Error evaluating variants

comment:4 Changed 7 years ago by mf2k (Frank Schima)

Resolution: → fixed
Status: new → closed

In c8004f0d089113cf65a1e08cc880955adb247853/macports-ports:

salt: Fix default_variants

  • Maintainer timeout

Fixes: #55016

Note: See TracTickets for help on using tickets.