gren: 0.4.5 -> 0.6.1

[robinheghan]

Context: I'm the lead developer of Gren. This is also my first contribution to nixpkgs, I apologize if I've missed something obvious.

This PR updates the Gren programming language to the latest released version.

Gren has become a little tricky to release since 0.5.0. The Gren compiler now consists of a (Haskell-based) backend and a (Gren/Node-based) frontend.

The frontend is bootstrapped, meaning it's written in Gren.

A release build of the frontend is commited to the official git repository, which makes things rather easy.

For the frontend to use the nix-built backend, a GREN_BIN environment variable must be set that points to this binary. If not set, the frontend will download a pre-built backend from github and install in XDG_CACHE. To set GREN_BIN, I've opted to use the wrapProgram utility function.

I've also added git to PATH using wrapProgram --suffix, as Gren's package manager relies on git being installed.

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/5512

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/5703

[TomaSajt]

I'll try to review this.

[robinheghan]

I'll try to review this.

Thank you!

I couldn't decide if I should've left your name in the maintainer list or not 🤷‍♂️

[TomaSajt]

I added a commit that cleans up the package:

It uses buildNpmPackage to not have to generate node_modules inside of the nixpkgs tree (which is not allowed).

buildNpmPackage correctly installs the binary as written inside package.json, though the wrapping still needed to be done with makeWrapper in postInstall.

I added extra meta fields.

I moved the backend package into passthru.backend so that it can be build separately if we want to.

Also, I improved the update.sh script.

Could you tell me how the versioning / tagging scheme works for gren? There's blob-{version} and there's ${version}.
Which one should we be using?

[TomaSajt]

If you've reviewed my changes, please, squash all of the commits in the PR.

The commit message should be gren: 0.4.5 -> 0.6.0 (maybe also update the PR title)

[robinheghan]

It uses buildNpmPackage to not have to generate node_modules inside of the nixpkgs tree (which is not allowed).

The frontend doesn't have any dependencies, and can run without node_modules. There should be no reason to use buildNpmPackage.

Could you tell me how the versioning / tagging scheme works for gren? There's blob-{version} and there's ${version}.
Which one should we be using?

{version} is the frontend
blob-{version} is the backend

The other changes looks like good improvements :)

[TomaSajt]

Alright I'll push another commit

[TomaSajt]

Alright, I pushed it.

[robinheghan]

I've squashed all the commits

[TomaSajt]

Oh, sorry, I forgot this PR also adds to you the maintainer list.

In that case, please create two commits instead:

• the first should be maintainers: add robinheghan which contains only the change to maintainer-list.nix
• the second one should be gren: 0.4.5 -> 0.6.0 with all other changes.

After that I think we're good to go.

[TomaSajt]

Good catch with the shebang of the script!

[robinheghan]

Are we just waiting for all the checks to complete, or are there other things that needs to be done before this can be merged?

[robinheghan]

@TomaSajt Updated to Gren 0.6.1 which was released today.

I couldn't get the nix-update command of update.hs to work, so I removed it, and updated the version and github hash in package.nix manually.

Is there anything else that needs to be done before this can be merged?

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/is-it-normal-for-ofborg-github-actions-to-stay-pending-forever/67505/1