Adam Hoka wrote:
Hi!PLIST_VARS+= nss pam .if !empty(PKG_OPTIONS:Mpam) PLIST.pam= defined .endif .if !empty(PKG_OPTIONS:Mnss) PLIST.nss= defined .endifCouldn't it be more automagical? Like, using PKG_SUPPORTED_OPTIONS instead of introducing PLIST_VARS, and using it like this: .if !empty(PKG_OPTIONS:Mpam) PLIST_UNHIDE+= pam .endif
This isn't just about PKG_OPTIONS, e.g.
PLIST_VARS+= cifsmount
.if ${OPSYS} == "Linux"
PLIST_VARS.cifsmount= defined
.endif
Of course, having a default of:
PLIST_VARS+= ${PKG_SUPPORTED_OPTIONS}
would be worth considering if that's the most common case, but I haven't
investigated whether that's true yet or not.
Cheers,
-- Johnny C. Lam