On 2020/03/14 15:29, Stuart Henderson wrote:
> On 2020/03/14 16:07, Klemens Nanni wrote:
> > Only the -zenmap package uses Python, -main is standalone.
> > 
> > This is another quirk I need to purge Python 2 while keeping useful
> > packages.
> > 
> > Feedback? OK?
> 
> >  MODLUA_SA= Yes
> >  MODLUA_VERSION=    5.3
> > +MODPY_RUNDEP=      No
> 
> Generally I would prefer to avoid MODPY_RUNDEP=No for a port which does
> actually have a run dep on python (just in a subpackage) and do things by
> setting RUN_DEPENDS-main instead for the lua dep...but lua.port.mk is a
> bit special with how it handles run dep's so let's not go there :)
> 
> So: could you add a "# added manually for -zenmap" or something next to
> the RUNDEP=No line please?
> 
> >  MODULES=   lang/python \
> >             lang/lua
> >  
> > @@ -53,7 +55,8 @@ LIB_DEPENDS-main=devel/pcre \
> >             ${MODLUA_LIB_DEPENDS} \
> >             security/libssh2
> >  
> > -RUN_DEPENDS-zenmap=        x11/py-gtk2 \
> > +RUN_DEPENDS-zenmap=        ${MODPY_RUN_DEPENDS} \
> > +                   x11/py-gtk2 \
> >                     databases/py-sqlite2 \
> >                     devel/desktop-file-utils \
> >                     net/nmap
> > 
> 

(otherwise OK)

Reply via email to