shen-sbcl: 3.0.3 -> 39.1

[hakujin]

Migrates to the new "S"-style Shen kernels which brings with them a new build system. Removes the (now deprecated) shen-sources package.

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.

[donovanglover]

@bsima As the author of the original PR, are you still interested in maintaining this package? I can't request you as a reviewer since you aren't in the NixOS organization.

[donovanglover]

Please create multiple commits to follow contributing guidelines. Make sure that the maintainer commit comes before adding yourself as a maintainer of the package.

• maintainers: add hakujin
• shen-sbcl: 3.0.3 -> 39.1
• shen-sbcl: add hakujin as maintainer

[hakujin]

@donovanglover is there anything else that needs to happen here? It looks like there is one stuck ofBorg check that has been queued for about 24 hours. Is this typical?

[donovanglover]

Pending CI checks are common and not a blocker for merges. Tools like nixpkgs-review-gha are often used to confirm that packages build successfully.

We still have to review other parts of the PR, which can take a while depending on how many people with the adequate knowledge are interested in it.

[donovanglover]

LGTM after changing the commit message to shen-sbcl: 3.0.3 -> 39.1. Also left some nits to align the package with conventions

[hakujin]

LGTM after changing the commit message to shen-sbcl: 3.0.3 -> 39.1. Also left some nits to align the package with conventions

Thanks for the feedback; I believe everything has been addressed and this is ready to go.

[donovanglover]

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 380807

Logs: https://github.com/donovanglover/nixpkgs-review-gha/actions/runs/13826754550

---

x86_64-linux

✅ 1 package built:

• shen-sbcl

---

aarch64-linux

✅ 1 package built:

• shen-sbcl

---

x86_64-darwin

✅ 1 package built:

• shen-sbcl

---

aarch64-darwin

✅ 1 package built:

• shen-sbcl

[donovanglover]

Tested and LGTM grats on your first contribution to nixpkgs!

Edit: Actually this isn't your first contribution but welcome to the maintainers list!