Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: added a nix flake #197

Draft
wants to merge 17 commits into
base: master
Choose a base branch
from
Draft

build: added a nix flake #197

wants to merge 17 commits into from

Conversation

teto
Copy link

@teto teto commented Jul 18, 2023

I would like to help the community contribute to luarocks.org (with a few issues in mind see nvim-neorocks/luarocks-tag-release#73) . As it stands, testing changes seems like an obstacle:

which is why I would like to submit a nix package. nix is a package manager that focuses on reproducibility, basically what works on my machine works on yours. Many technologies promise this but nix delivers.

Here is my attempt at it (in my constrained time so anyone feel free to pick up).

  • dont use absolute location towards luajit in Makefile
  • how to set an alternate postgresql port ?
  • add comment in nginx.conf about how variables are replaced
  • improve instructions

teto and others added 5 commits August 17, 2023 18:24
run with nix develop

we need nixpkgs to export some variables to help luarocks
assume "coffee" to be in PATH
wip

adding an input.csv to regenerate nix code from

added README

fixed lapis build ! nix build .#lapis works

and put it in the devShell so one can straight run "lapis server"

ok

wip

we need to use a lua5.1 env since openresty relies on luajit, else you
get missing symbol errors.

copying nginx.conf to service

mount luarocks-site folder for its config in the VM. But needs to add lapis.coroutine to luaEnv

more dependencies
it's not portable. It's best to assume luajit already in PATH
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant