tech-pkg archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

multiversion lua, buildlink3 and graphviz



Since the recent multiversion lua reorganisation, there seems to be a little bit of fallout. One package, graphviz, provides lua scripts, but looks for 'lua' during configure.

From a discussion off-list with wiz@ concerning graphviz

well, this worked for me (complete build/package):
>diff --git a/graphics/graphviz/options.mk b/graphics/graphviz/options.mk
>index 0ccd5ef..586f4cb 100644
>--- a/graphics/graphviz/options.mk
>+++ b/graphics/graphviz/options.mk
>@@ -114,6 +114,7 @@ USING_SWIG= no
> .if !empty(PKG_OPTIONS:Mlua)
> USING_SWIG=    yes
> .include "../../lang/lua/buildlink3.mk"
>+.include "../../lang/lua/tool.mk"
> PLIST.lua=             yes
> .else
> CONFIGURE_ARGS+=       --disable-lua

I believe, though, that lua/buildlink3.mk should probably directly include
lua/tool.mk... just to avoid this sort of hassle!

Perhaps lua/buildlink3.mk could add tools.mk after

> .include "../../lang/lua/luaversion.mk"
> .include "${LUA_PKGSRCDIR}/buildlink3.mk"

in order to avoid this particular, perhaps commonplace, problem..



Home | Main Index | Thread Index | Old Index