fix(luarocks): rely on luarocks wrapping for luarocks package

Author: BirdeeHub

.github/rockspec.template

@@ -36,8 +36,10 @@ build = {
   copy_directories = $copy_directories,
   build_variables = {
     LUA_INCDIR="$(LUA_INCDIR)",
+    LIBFLAG="$(LIBFLAG)",
   },
   install_variables = {
+    WRAP = "false",
     LIBDIR="$(LIBDIR)",
     BINDIR="$(BINDIR)",
     LUADIR="$(LUADIR)",

Makefile

@@ -100,12 +100,16 @@ ifdef LUADIR
 endif
 ifdef BINDIR
 	@mkdir -p "$(BINDIR)";
+ifneq ($(WRAP),false)
 ifeq ($(filter /%,$(LUA)),)
 	@echo '$(FIX_SHEBANG)' | $(LUA) - "/usr/bin/env $(LUA)" "$(abspath $(LIBDIR))" > "$(BINDIR)/tomlua"
 else
 	@echo '$(FIX_SHEBANG)' | $(LUA) - "$(LUA)" "$(abspath $(LIBDIR))" > "$(BINDIR)/tomlua"
 endif
 	@cat "$(SRC)/bin/tomlua" >> "$(BINDIR)/tomlua";
+else
+	@cat "$(SRC)/bin/tomlua" > "$(BINDIR)/tomlua";
+endif
 	@chmod +x "$(BINDIR)/tomlua";
 	@echo "Installed binary to $(BINDIR)";
 endif

flake.nix

@@ -26,31 +26,33 @@
     };
     APPNAME = "tomlua";
     overlay = final: prev: let
-      luaCallPackageFn = { toLuaModule, lua, }:
-        toLuaModule (lua.stdenv.mkDerivation (finalAttrs: {
+      luaCallPackageFn = { buildLuarocksPackage, }:
+        buildLuarocksPackage {
           pname = APPNAME;
-          version = "${(finalAttrs.passthru.luaModule or lua).luaversion}.dev";
+          version = "scm-1";
           src = self;
           doCheck = false;
-          __structuredAttrs = true;
-          strictDeps = true;
-          propagatedBuildInputs = [ (finalAttrs.passthru.luaModule or lua) ];
-          env = {
-            LUA = (finalAttrs.passthru.luaModule or lua).interpreter;
-            LUA_INC = (finalAttrs.passthru.luaModule or lua) + "/include";
-            BINDIR = placeholder "out" + "/bin";
-            LIBDIR = placeholder "out" + "/lib/lua/" + (finalAttrs.passthru.luaModule or lua).luaversion;
-            LUADIR = placeholder "out" + "/share/lua/" + (finalAttrs.passthru.luaModule or lua).luaversion;
-            LIBFLAG = if (finalAttrs.passthru.luaModule or lua).stdenv.isDarwin then "-bundle -undefined dynamic_lookup" else "-shared";
-          };
+          checkPhase = ''
+            runHook preCheck
+            runHook postCheck
+          '';
+          installCheckPhase = ''
+            runHook preInstallCheck
+            make test
+            runHook postInstallCheck
+          '';
+          postConfigure = ''
+            chmod +w "$rockspecFilename"
+            echo "deploy = { wrap_bin_scripts = false, }" >> "$rockspecFilename"
+          '';
           meta = {
             mainProgram = "tomlua";
             maintainers = [ lib.maintainers.birdee ];
             license = lib.licenses.mit;
             homepage = "https://github.com/BirdeeHub/tomlua";
             description = "Speedy toml parsing for lua, implemented in C";
           };
-        }));
+        };
       # lua5_1 = prev.lua5_1.override { packageOverrides };
       l_pkg_main = builtins.mapAttrs (
         n: _: (prev.lib.attrByPath [ n "override" ] null prev) {

tomlua-scm-1.rockspec

@@ -21,8 +21,10 @@ build = {
    type = "make",
    build_variables = {
       LUA_INCDIR="$(LUA_INCDIR)",
+      LIBFLAG="$(LIBFLAG)",
    },
    install_variables = {
+      WRAP = "false",
       LIBDIR="$(LIBDIR)",
       BINDIR="$(BINDIR)",
       LUADIR="$(LUADIR)",
@@ -33,16 +35,3 @@ test = {
    type = "command",
    command = "make test",
 }
-
--- How do I specify CFLAGS -O3 and maybe -flto here?
--- build = {
---    type = "builtin",
---    modules = {
---       tomlua = {
---           "./src/tomlua.c",
---           "./src/decode.c",
---           "./src/encode.c",
---           "./src/dates.c",
---       },
---    }
--- }