README: fix node name look-up #61

Closed
Gerg-L wants to merge 1 commit from Gerg-L/master into master
Gerg-L commented 2023-10-24 01:08:08 +00:00 (Migrated from github.com)

Using the example in the README will not use the correct flake-compat url/hash if another input with a name before "flake-compat" in ASCII ordering has a "flake-compat" input

I made a minimal reproducible example repo

Tested via:

current version:

nix-shell -p '(import (builtins.fetchTarball "https://github.com/Gerg-L/flake-compat-example/archive/98c5dae31e253ee2a4eb5e19867d43f790e95425.tar.gz")).packages.${builtins.currentSystem}.default'

after this PR:

nix-shell -p '(import (builtins.fetchTarball "https://github.com/Gerg-L/flake-compat-example/archive/6023171dd1df408b880007e09716227beab8016a.tar.gz")).packages.${builtins.currentSystem}.default'
Using the example in the README will not use the correct flake-compat url/hash if another input with a name before "flake-compat" in ASCII ordering has a "flake-compat" input I made a minimal reproducible example [repo](https://github.com/Gerg-L/flake-compat-example/tree/98c5dae31e253ee2a4eb5e19867d43f790e95425) Tested via: ### current version: ```bash nix-shell -p '(import (builtins.fetchTarball "https://github.com/Gerg-L/flake-compat-example/archive/98c5dae31e253ee2a4eb5e19867d43f790e95425.tar.gz")).packages.${builtins.currentSystem}.default' ``` ### after this PR: ```bash nix-shell -p '(import (builtins.fetchTarball "https://github.com/Gerg-L/flake-compat-example/archive/6023171dd1df408b880007e09716227beab8016a.tar.gz")).packages.${builtins.currentSystem}.default' ````
jade closed this pull request 2024-05-03 02:57:54 +00:00

Pull request closed

Sign in to join this conversation.
No description provided.