Skip to content

Nix dev flake#937

Merged
Try merged 7 commits into
Try:masterfrom
pancakedevourer:nix-flake
May 21, 2026
Merged

Nix dev flake#937
Try merged 7 commits into
Try:masterfrom
pancakedevourer:nix-flake

Conversation

@pancakedevourer
Copy link
Copy Markdown
Contributor

@pancakedevourer pancakedevourer commented May 18, 2026

Added a Nix dev flake, for muuuch easier compilation on NixOS (or other systems with Nix).
Also updated the README to mention using the flake :)

@Try
Copy link
Copy Markdown
Owner

Try commented May 19, 2026

Hi, @pancakedevourer and thanks for PR!
Sorry, github is very laggy for me right now, I'll have a proper look in maybe day/two.

Small question for now, can flake files be located in a folder(./linux/flake), to avoid crowd in root directory?

@pancakedevourer
Copy link
Copy Markdown
Contributor Author

pancakedevourer commented May 19, 2026 •
edited
Loading

Oh sure, that's a good idea :)
Also, compiling stuff with many libraries on NixOS, without a flake, is quite hard, because it puts everything in /nix/store, and no compiler/build system will ever find them. And a flake such as this is very convienient, as one also oesn't have to install the dependencies system-wide :)

Try
Try approved these changes May 21, 2026
View reviewed changes
Copy link
Copy Markdown
Owner

@Try Try left a comment

Choose a reason for hiding this comment

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

LGTM

Small question: Am I correct to assume that this setup is essentially set-and-forget kind, and wont require adjustments on new releases?

Comment thread linux/flake/flake.nix Outdated
alsa-lib
libX11
libXcursor
libglvnd
Copy link
Copy Markdown
Owner

@Try Try May 21, 2026

Choose a reason for hiding this comment

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

I don't think we need libglvnd. There is no opengl support after all...
PS:
just noticed, that it's mentioned in the readme - probably by mistake

Copy link
Copy Markdown
Contributor Author

@pancakedevourer pancakedevourer May 21, 2026

Choose a reason for hiding this comment

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

Lol :D, I'll remove it. Quite definitely won't require adjustments :)

@Try Try merged commit 2d5f741 into Try:master May 21, 2026
3 of 7 checks passed
@Try
Copy link
Copy Markdown
Owner

Try commented May 21, 2026

Merged, thanks!

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

Reviewers

@Try Try Try approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pancakedevourer @Try