Currently, supplying any commands more than 156 causes stack corruption due to Lua freeing the pointer to the string.
With enough luck, you might get to run the program, but it will cause a double free error.

With the latest commit, the parent's environment variables are passed to the child process
when the env table is specified. This matches POSIX's behavior.

Will test this on windows in the near future, but otherwise, even if it's a bit longer/larger than I would hope for, looks good.

On windows, running the following code causes an issue: cannot add environment variable: unknown error (-1) in the lite-xl error log.

local p1 = process.start({ "./a.exe" }, { env = { TEST = "Keaops" } })
while p1:running() do
  local output = p1:read_stdout()
  if output ~= "" and output ~= nil then
    print("OUTPUT", output)
  end
end
print(p1:returncode())

It didn't before; runs fine on the original code.

On windows, running the following code causes an issue: cannot add environment variable: unknown error (-1) in the lite-xl error log.

local p1 = process.start({ "./a.exe" }, { env = { TEST = "Keaops" } })
while p1:running() do
  local output = p1:read_stdout()
  if output ~= "" and output ~= nil then
    print("OUTPUT", output)
  end
end
print(p1:returncode())

It didn't before; runs fine on the original code.

I cannot replicate this issue; but I believe I found the cause and committed a fix. Please test it.

Yes, this now runs. Confirmed. Will do some more testing, and if it all looks good, will merge.

OK, reviewed code a bit more; bit weird about the allocations being very different on Windows vs. everywhere else, but Windows is windows, and I seem to remember something about needing contiguous blocks or whatever; so I'm sure that's fine. Merging.

OK, reviewed code a bit more; bit weird about the allocations being very different on Windows vs. everywhere else, but Windows is windows, and I seem to remember something about needing contiguous blocks or whatever; so I'm sure that's fine. Merging.

Yeah, Windows environment block expects a long "string" of each environment variables delimited by NUL.