Skip to content

Create shell.nix - #1199

Open
DewJunkie wants to merge 2 commits into
munificent:masterfrom
DewJunkie:master
Open

Create shell.nix#1199
DewJunkie wants to merge 2 commits into
munificent:masterfrom
DewJunkie:master

Conversation

@DewJunkie

Copy link
Copy Markdown

Add shell.nix to install dart2 and zulu, nix will already have a c compiler

Add shell.nix to install dart2 and zulu, nix will already have a c compiler
@DewJunkie

Copy link
Copy Markdown
Author

I'm not sure if this is interesting or not, but the idea is that the build becomes batteries included on nix. I ended up creating it, because when I installed dart, it was 3.x, and every time I have to figure out how to install an older version of something, I end up re-learning how. If we wanted to take it all the way, then it could be made into a Pure shell, and all the dependencies (besides what is in "make get") would need to be specified. Taking it even further, but I think that would be a step back for other platforms, would be to include everything that make get does.

@bernborgess

Copy link
Copy Markdown

Looks good! Thanks for the contribution! It would be better if we could patch it to be null-safe tho...

@DewJunkie

Copy link
Copy Markdown
Author

ok, I'll work on that, I'll also update this script to that is works with
nix-shell --pure.
I'll try and work on the null-safety for Dart3.

@bernborgess

Copy link
Copy Markdown

ok, I'll work on that, I'll also update this script to that is works with
nix-shell --pure.
I'll try and work on the null-safety for Dart3.

Cool, want any help with that? Docs should explain what to do, but I don't know if it's a lot or just a couple changes

> allow debug C builds when tooling injects _FORTIFY_SOURCE
@DewJunkie

DewJunkie commented May 16, 2026

Copy link
Copy Markdown
Author

It wasn't too bad, I made this story to track it
#1212
I think reviewing this should be reasonably quick and safe, When I'm done testing/reviewing locally I'll make another PR with those changes. Having a nix-shell, but I guess a container would be equally good, made this go pretty quick and safe

@DewJunkie DewJunkie mentioned this pull request Jun 1, 2026
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