Skip to content
This repository was archived by the owner on Jun 19, 2026. It is now read-only.

add: more complete shell.nix - #2614

Open
Ben-PH wants to merge 1 commit into
hannobraun:mainfrom
Ben-PH:shell_fixes
Open

add: more complete shell.nix#2614
Ben-PH wants to merge 1 commit into
hannobraun:mainfrom
Ben-PH:shell_fixes

Conversation

@Ben-PH

@Ben-PH Ben-PH commented Mar 31, 2026

Copy link
Copy Markdown

Disclaimer: changes made under the output of an LLM

Evidently my sysconfig is missing some things. compiling, i got this error:

➜ cargo run -p cuboid
error: rustc 1.91.1 is not supported by the following packages:
  wgpu@28.0.0 requires rustc 1.92
  wgpu@28.0.0 requires rustc 1.92
  wgpu@28.0.0 requires rustc 1.92
Either upgrade rustc or select compatible dependency versions with
`cargo update <name>@<current-ver> --precise <compatible-ver>`
where `<compatible-ver>` is the latest version supporting rustc 1.91.1

The fix was to add rustup to packages (including the local dir use)

I then had this error:

     Running `target/debug/cuboid`
Error: Error displaying model

Caused by:
    0: Error initializing event loop
    1: os error at /home/ben/rust/fornjot/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/winit-0.30.13/src/platform_impl/linux/mod.rs:788: Failed to load one of xlib's shared libraries

Which was fixed with the xorg additions to libpath

@Ben-PH
Ben-PH requested a review from hannobraun as a code owner March 31, 2026 09:45

@hannobraun hannobraun left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you for this pull request!

I think this makes sense, as it makes the repository more accessible for other Nix users, while the previous version was only written to plug the holes in my personal setup.

The RUST_HOME/CARGO_HOME part doesn't work though. I'm not so sure it makes sense on principle, as every project that is set up like this will duplicate the same files instead of sharing them. If we wanted to do it like that, these directories would need to go into .gitignore though.

Disclaimer: changes made under the output of an LLM

Thank you, I appreciate the heads-up. Since this is a small and understandable change, I'll accept it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants