diff options
author | Eelco Dolstra <edolstra@gmail.com> | 2019-08-30 16:38:27 +0200 |
---|---|---|
committer | Eelco Dolstra <edolstra@gmail.com> | 2019-08-30 16:38:27 +0200 |
commit | 80c36d4562af71a90c67b3adb886a1003834890e (patch) | |
tree | 82be5f00ade2b2f8ba79bb150bcd5a91965d12c1 | |
parent | 30ccf4e52d31ea0f1531feb8cccdc0314a41265d (diff) |
Remove 'name' attribute from flakes
This is no longer needed since flakes are given an identity in the
'inputs' attribute.
-rw-r--r-- | flake.lock | 7 | ||||
-rw-r--r-- | flake.nix | 2 | ||||
-rw-r--r-- | src/libexpr/flake/flake.cc | 6 | ||||
-rw-r--r-- | src/libexpr/flake/flake.hh | 1 | ||||
-rw-r--r-- | src/nix/flake.cc | 11 | ||||
-rw-r--r-- | tests/flakes.sh | 20 |
6 files changed, 6 insertions, 41 deletions
diff --git a/flake.lock b/flake.lock index 8072152ca..fa5649c03 100644 --- a/flake.lock +++ b/flake.lock @@ -1,13 +1,10 @@ { "inputs": { "nixpkgs": { - "id": "nixpkgs", "inputs": {}, - "narHash": "sha256-AndIaZrFFIT+VFhVtQHsS90I5SWfjTDTxzs9Hx9ZxZA=", - "nonFlakeInputs": {}, - "uri": "github:edolstra/nixpkgs/9bd2e2c96ddeec64e9ad37540412263fdb78458d" + "narHash": "sha256-TrLhI3xPkTTznE9gcMHhkHirGNN7N02zM4DxJ/U3WRs=", + "uri": "github:edolstra/nixpkgs/24bf27fc215e8300877dfa1c426b9966bbfbd150" } }, - "nonFlakeInputs": {}, "version": 2 } @@ -1,6 +1,4 @@ { - name = "nix"; - description = "The purely functional package manager"; edition = 201909; diff --git a/src/libexpr/flake/flake.cc b/src/libexpr/flake/flake.cc index ce5dd55a2..ba337cf97 100644 --- a/src/libexpr/flake/flake.cc +++ b/src/libexpr/flake/flake.cc @@ -239,11 +239,6 @@ Flake getFlake(EvalState & state, const FlakeRef & flakeRef) } else throw Error("flake '%s' lacks attribute 'edition'", flakeRef); - if (auto name = vInfo.attrs->get(state.sName)) - flake.id = state.forceStringNoCtx(*(**name).value, *(**name).pos); - else - throw Error("flake '%s' lacks attribute 'name'", flakeRef); - if (auto description = vInfo.attrs->get(state.sDescription)) flake.description = state.forceStringNoCtx(*(**description).value, *(**description).pos); @@ -292,7 +287,6 @@ Flake getFlake(EvalState & state, const FlakeRef & flakeRef) for (auto & attr : *vInfo.attrs) { if (attr.name != sEdition && attr.name != sEpoch && - attr.name != state.sName && attr.name != state.sDescription && attr.name != sInputs && attr.name != sOutputs) diff --git a/src/libexpr/flake/flake.hh b/src/libexpr/flake/flake.hh index 84bce9b5a..8f05e9799 100644 --- a/src/libexpr/flake/flake.hh +++ b/src/libexpr/flake/flake.hh @@ -67,7 +67,6 @@ struct FlakeInput struct Flake { - FlakeId id; FlakeRef originalRef; std::string description; SourceInfo sourceInfo; diff --git a/src/nix/flake.cc b/src/nix/flake.cc index aab29b626..4129ef323 100644 --- a/src/nix/flake.cc +++ b/src/nix/flake.cc @@ -103,7 +103,6 @@ static void sourceInfoToJson(const SourceInfo & sourceInfo, nlohmann::json & j) static void printFlakeInfo(const Flake & flake) { - std::cout << fmt("ID: %s\n", flake.id); std::cout << fmt("Description: %s\n", flake.description); std::cout << fmt("Edition: %s\n", flake.edition); printSourceInfo(flake.sourceInfo); @@ -112,7 +111,6 @@ static void printFlakeInfo(const Flake & flake) static nlohmann::json flakeToJson(const Flake & flake) { nlohmann::json j; - j["id"] = flake.id; j["description"] = flake.description; j["edition"] = flake.edition; sourceInfoToJson(flake.sourceInfo, j); @@ -120,12 +118,6 @@ static nlohmann::json flakeToJson(const Flake & flake) } #if 0 -static void printNonFlakeInfo(const NonFlake & nonFlake) -{ - std::cout << fmt("ID: %s\n", nonFlake.alias); - printSourceInfo(nonFlake.sourceInfo); -} - // FIXME: merge info CmdFlakeInfo? struct CmdFlakeDeps : FlakeCommand { @@ -148,9 +140,6 @@ struct CmdFlakeDeps : FlakeCommand auto resFlake = std::move(todo.front()); todo.pop(); - for (auto & nonFlake : resFlake.nonFlakeDeps) - printNonFlakeInfo(nonFlake); - for (auto & info : resFlake.flakeDeps) { printFlakeInfo(info.second.flake); todo.push(info.second); diff --git a/tests/flakes.sh b/tests/flakes.sh index 7c16304ec..8594a4c55 100644 --- a/tests/flakes.sh +++ b/tests/flakes.sh @@ -29,8 +29,6 @@ done cat > $flake1Dir/flake.nix <<EOF { - name = "flake1"; - edition = 201909; description = "Bla bla"; @@ -48,8 +46,6 @@ git -C $flake1Dir commit -m 'Initial' cat > $flake2Dir/flake.nix <<EOF { - name = "flake2"; - edition = 201909; description = "Fnord"; @@ -65,8 +61,6 @@ git -C $flake2Dir commit -m 'Initial' cat > $flake3Dir/flake.nix <<EOF { - name = "flake3"; - edition = 201909; description = "Fnord"; @@ -118,12 +112,12 @@ EOF (( $(nix flake list --flake-registry $registry | wc -l) == 5 )) # Test 'nix flake info'. -nix flake info --flake-registry $registry flake1 | grep -q 'ID: *flake1' +nix flake info --flake-registry $registry flake1 | grep -q 'URI: .*flake1.*' # Test 'nix flake info' on a local flake. -(cd $flake1Dir && nix flake info --flake-registry $registry) | grep -q 'ID: *flake1' -(cd $flake1Dir && nix flake info --flake-registry $registry .) | grep -q 'ID: *flake1' -nix flake info --flake-registry $registry $flake1Dir | grep -q 'ID: *flake1' +(cd $flake1Dir && nix flake info --flake-registry $registry) | grep -q 'URI: .*flake1.*' +(cd $flake1Dir && nix flake info --flake-registry $registry .) | grep -q 'URI: .*flake1.*' +nix flake info --flake-registry $registry $flake1Dir | grep -q 'URI: .*flake1.*' # Test 'nix flake info --json'. json=$(nix flake info --flake-registry $registry flake1 --json | jq .) @@ -183,8 +177,6 @@ rm $flake3Dir/flake.nix cat > $flake3Dir/flake.nix <<EOF { - name = "flake3"; - edition = 201909; description = "Fnord"; @@ -236,8 +228,6 @@ rm $flake3Dir/flake.nix cat > $flake3Dir/flake.nix <<EOF { - name = "flake3"; - edition = 201909; inputs = { @@ -305,8 +295,6 @@ rm $flake3Dir/flake.nix cat > $flake3Dir/flake.nix <<EOF { - name = "flake3"; - edition = 201909; inputs = { |