Skip to content

Conversation

@HU90m
Copy link
Member

@HU90m HU90m commented Jul 1, 2025

Saw NixOS/nix#12421 had made it's way into the latest release, so wanted to update sonata-software.

This will require users be on newer versions of the Nix CLI.

HU90m added 2 commits July 1, 2025 10:56
Nix now shows a warning for apps that don't have descriptions.
```
warning: app 'apps.x86_64-linux.lint-cpp' lacks attribute 'meta.description'
```
In the latest release of Nix, one can now declare that they'd like
submodules included within the flake files inputs.
@HU90m HU90m requested review from engdoreis and nbdd0121 July 1, 2025 08:06
@nbdd0121
Copy link
Contributor

nbdd0121 commented Jul 1, 2025

Which version of nix does this require?

@marnovandermaas
Copy link
Contributor

And as a follow up is that supported in the latest determinate systems install: https://lowrisc.github.io/sonata-software/doc/getting-started.html#installing-nix

@HU90m
Copy link
Member Author

HU90m commented Jul 1, 2025

It requires Nix version 2.27. (The latest version is 2.28.3)

Yes it looks like the determinate system is at least on version 2.28.2

Copy link
Contributor

@engdoreis engdoreis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a nice tidy up, thanks @HU90m.
@nbdd0121 do you have any concern regarding the version, or can we merge this PR?

@marnovandermaas marnovandermaas merged commit 722cdea into lowRISC:main Jul 8, 2025
1 check passed
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.

4 participants