[BUG] hash mismatch on nix flake

[yari-dog]

Prerequisites

Please check the following before continuing:

• I have searched for existing issues, including closed issues to ensure this is not a duplicate.

this is sorta a duplicate of #73 but only if you think about it too much

• This is a issue for the Luna Client and not a Plugin
• I am running the latest Luna Client version (See latest release)

Environment

• OS: Linux - NixOS
• Client Version: Current flake

Description

injection.nix in commit c554438 is specifying the wrong hash for pnpm-deps

error: hash mismatch in fixed-output derivation '/nix/store/111v079xgfdkmpk7ikak8bj8ishlkyfc-TidaLuna-pnpm-deps.drv':
    specified: sha256-pHIY4Ie66ZVwEne/4RmY2QvsRWcnfsl2kv3CDXcqVrg=
    got:    sha256-Oj34rQbKbsHnqPdVv+ti8z+gZTT+VOsDxg/MQ22sLRQ=

Steps to Reproduce

Steps to reproduce the behavior:

1. update flake
2. rebuild

[frostplexx]

For me on macOS the hash sha256-pHIY4Ie66ZVwEne/4RmY2QvsRWcnfsl2kv3CDXcqVrg= works and sha256-Oj34rQbKbsHnqPdVv+ti8z+gZTT+VOsDxg/MQ22sLRQ= doesn't.
Are pnpm hashes supposed to differ between NixOS and macOS?

[yari-dog]

seems like it - NixOS/nix#6837 !
pretty interesting, not sure why that would be the case ? i dont know enough about pnpm to speculate aha

[yari-dog]

ive just reverted my flake.lock back to

      "locked": {
        "lastModified": 1771024355,
        "narHash": "sha256-yGKWWql6FZp/IvMgdQGOlze27FLpELDuYFR8Wn+DOnM=",
        "owner": "Inrixia",
        "repo": "TidaLuna",
        "rev": "ffce6283e54a7b7bcf9bd12112956a84d5e34834",
        "type": "github"
      },

in the meantime , i assume figuring out a way to circumvent that issue is gonna be annoying so i thought i'd just post the last known working one for anyone else so u get a bit more time to cook

[frostplexx]

Doing something like

  pnpmDeps = fetchPnpmDeps {
    inherit pname src version;
    fetcherVersion = 1;
    hash = if stdenv.isDarwin 
                then "sha256-pHIY4Ie66ZVwEne/4RmY2QvsRWcnfsl2kv3CDXcqVrg=" 
                else "sha256-Oj34rQbKbsHnqPdVv+ti8z+gZTT+VOsDxg/MQ22sLRQ=";
  };

will probably work. Not sure if its best practice though

[frostplexx]

@yari-dog I created a draft PR #141 that applies the fix I mentioned. Are you able to test it?

[yari-dog]

@yari-dog I created a draft PR #141 that applies the fix I mentioned. Are you able to test it?

aye ill give it a go , wont be a minute

[yari-dog]

works fine, aye ^^

[frostplexx]

That's great to hear. Short of completely splitting the build process for Darwin and NixOS this seems to be the simplest and cleanest approach. I'll open my draft for review then.

[yari-dog]

sounds like a plan, thank you for doing that so quickly !
there's been some other packages that i know of that have had weird pnpm hash issues so i'd hazard a guess that this darwin-linux thing is behind it

[Inrixia]

@ojsef39 for visibility