On Fri, Apr 22, 2016 at 09:14:17AM +0200, Florian Stinglmayr wrote: > > Thinking about it some more "lua-yaml" makes more sense than > > "lua-lyaml". > > > > There is another project called lua-yaml [1], and there's even another > just called "yaml" for Lua [2]. I thought it might be important to clear > up by name which one we have in the ports. > > But I am okay with all the other changes. > > Thanks for submitting it. > > Regards, > Florian > > [1] http://luaforge.net/projects/luayaml/ > [2] http://doc.lubyk.org/yaml.html
Some time ago when needing to use yaml from lua I ended up with the same project as this port. Debian calls their lyaml package lua-yaml so it isn't without precedent https://packages.debian.org/sid/lua-yaml > > > This also fixes the distfile name to include a version. > > > > diff -upr lua-lyaml.orig/Makefile lua-yaml/Makefile > > --- lua-lyaml.orig/Makefile Fri Apr 22 13:14:20 2016 > > +++ lua-yaml/Makefile Fri Apr 22 14:09:56 2016 > > @@ -3,9 +3,7 @@ > > COMMENT = Lua library for parsing and emitting YAML > > V = 6.0 > > > > -DISTNAME = lua-${GH_PROJECT} > > -PKGNAME = ${DISTNAME}-${V} > > -WRKDIST = ${WRKDIR}/${GH_PROJECT}-${GH_TAGNAME} > > +PKGNAME = lua-yaml-${V} > > > > GH_ACCOUNT = gvvaughan > > GH_PROJECT = lyaml > > @@ -18,15 +16,16 @@ MAINTAINER = Florian Stinglmayr <flor...@n0la.org> > > # MIT > > PERMIT_PACKAGE_CDROM = Yes > > > > +WANTLIB = yaml-0 > > MODULES = lang/lua > > > > -BUILD_DEPENDS = devel/libyaml > > +LIB_DEPENDS = devel/libyaml > > > > CONFIGURE_STYLE = gnu > > -CONFIGURE_ENV = CPPFLAGS="-I${PREFIX}/include -L${PREFIX}/lib" \ > > - LUA="${MODLUA_BIN}" > > - > > -LIBTOOL_FLAGS += --tag=disable-static > > +CONFIGURE_ENV = CPPFLAGS="-I${PREFIX}/include" \ > > + LDFLAGS="-L${PREFIX}/lib" \ > > + LUA=${MODLUA_BIN} \ > > + LUA_INCLUDE=-I${MODLUA_INCL_DIR} > > > > # Test requires specl, specl requires lyaml. > > NO_TEST = Yes > > diff -upr lua-lyaml.orig/distinfo lua-yaml/distinfo > > --- lua-lyaml.orig/distinfo Fri Apr 22 13:14:20 2016 > > +++ lua-yaml/distinfo Fri Apr 22 14:09:56 2016 > > @@ -1,2 +1,2 @@ > > -SHA256 (lua-lyaml.tar.gz) = nXz3TXdpmf9vdYxWnVIC/12h8wPG9CKdO0H3HNOj56c= > > -SIZE (lua-lyaml.tar.gz) = 475288 > > +SHA256 (lyaml-release-v6.0.tar.gz) = > > nXz3TXdpmf9vdYxWnVIC/12h8wPG9CKdO0H3HNOj56c= > > +SIZE (lyaml-release-v6.0.tar.gz) = 475288
