Skip to content

shell.nix: provide adafruit-nrfutil#2251

Open
mweinelt wants to merge 1 commit intomeshcore-dev:mainfrom
mweinelt:nix-shell-adafruit-nrfutil
Open

shell.nix: provide adafruit-nrfutil#2251
mweinelt wants to merge 1 commit intomeshcore-dev:mainfrom
mweinelt:nix-shell-adafruit-nrfutil

Conversation

@mweinelt
Copy link
Copy Markdown

@mweinelt mweinelt commented Apr 4, 2026

This is marked unfree in nixpkgs upstream, which is something we need to explicitly allow in our shell.

Then we need to permit CVE-2024-23342 in ecdsa, a timing security issue which upstream does not plan to resolve, but isn't relevant to the use of adafruit-nrfutil at all.

The switch to mkShellNoCC makes the shell closure smaller, since it does not include the nixpkgs stdenv.

Renamed from default.nix, as shell.nix is the cleaner name for when the entrypoint is a nix shell. This does not require changes to the direnv configuration.

This increases the shell closure from 21.55 MiB to 21.66 MiB.

cc @neutralinsomniac

This is marked unfree in nixpkgs upstream, which is something we need to
explicitly allow in our shell.

Then we need to permit CVE-2024-23342 in ecdsa, a timing security issue
which upstream does not plan to resolve, but isn't relevant to the use of
adafruit-nrfutil at all.

The switch to `mkShellNoCC` makes the shell closure smaller, since it
does not include the nixpkgs stdenv.

Renamed from default.nix, as shell.nix is the cleaner name for when the
entrypoint is a nix shell. This does not require changes to the direnv
configuration.
@neutralinsomniac
Copy link
Copy Markdown
Contributor

neutralinsomniac commented Apr 4, 2026 via email

@neutralinsomniac
Copy link
Copy Markdown
Contributor

Looks good to me and worked fine!

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.

2 participants