NixOS

[Paizo-dev]

Hi John,

I've been testing OkShell on NixOS and noticed a few technical omissions in the current Nix derivation that prevent the project from building "out of the box."

To improve reproducibility and fix the build process for Nix users, the following changes are required in package.nix:

1. Missing System Dependencies

The current derivation is missing several system libraries required by the Rust crates:

systemd (libudev): Required by libudev-sys. Without it, pkg-config fails to find libudev.pc.

alsa-lib: Required by alsa-sys for audio/notification features.

libclang: The clang-sys crate needs libclang and the LIBCLANG_PATH environment variable to be explicitly set in the Nix environment to find C headers.

2. Doctest Failures in okshell-gamma

The doctests for okshell-gamma (specifically in crates/okshell-gamma/src/service.rs) are currently failing because glib is not being properly linked during the Cargo test phase within the Nix sandbox.

Recommended Changes

I suggest updating the buildInputs and environment configuration as follows:

buildInputs = [

... existing dependencies ...

pkgs.systemd
pkgs.alsa-lib
pkgs.llvmPackages.libclang.lib
pkgs.glib
];

Required for bindgen/clang-sys

LIBCLANG_PATH = "${pkgs.llvmPackages.libclang.lib}/lib";

[JohnOberhauser]

I made your suggested changes, let me know if there are still issues.

b6974e1

[Paizo-dev]

Hi John,
Thanks for the quick update! I've tested it again on NixOS by adding it to my flake, and it now builds and installs without any issues.

However, I found a small edge case: while it works perfectly on my laptop, it failed to start on my desktop PC. The error was due to okshell looking for upower and power-profiles-daemon services, which aren't always active or installed on desktops by default. I had to manually enable those services in my NixOS configuration to get it running.

Maybe we could consider making those dependencies optional or adding a check so the app doesn't crash if they aren't present?
Thanks for your support!

[JohnOberhauser]

@Paizo-dev Does this PR make sense as a fix? #9

[Paizo-dev]

Hi John, that’s what NixOS expects to find. In the previous version, I had to add them manually to get it to compile. Today I've been reviewing what else is needed to unlock 100% of OkShell's features, so I’ll be adding more usage data in the next few hours. I'm attaching a screenshot for you.