Skip to content

Commit

Permalink
fixup! nix shell: reflect command line order in PATH order
Browse files Browse the repository at this point in the history
  • Loading branch information
cole-h committed Dec 21, 2023
1 parent e5d1a05 commit 858cb92
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 17 deletions.
15 changes: 0 additions & 15 deletions src/libcmd/built-path.cc
Original file line number Diff line number Diff line change
Expand Up @@ -58,21 +58,6 @@ StorePathSet BuiltPath::outPaths() const
);
}

StorePaths BuiltPath::outPaths2() const
{
return std::visit(
overloaded{
[](const BuiltPath::Opaque & p) { return StorePaths{p.path}; },
[](const BuiltPath::Built & b) {
StorePaths res;
for (auto & [_, path] : b.outputs)
res.push_back(path);
return res;
},
}, raw()
);
}

SingleDerivedPath::Built SingleBuiltPath::Built::discardOutputPath() const
{
return SingleDerivedPath::Built {
Expand Down
1 change: 0 additions & 1 deletion src/libcmd/built-path.hh
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,6 @@ struct BuiltPath : _BuiltPathRaw {
}

StorePathSet outPaths() const;
StorePaths outPaths2() const;
RealisedPath::Set toRealisedPaths(Store & store) const;

nlohmann::json toJSON(const StoreDirConfig & store) const;
Expand Down
2 changes: 1 addition & 1 deletion src/libcmd/installables.cc
Original file line number Diff line number Diff line change
Expand Up @@ -737,7 +737,7 @@ StorePaths Installable::toStorePaths(
{
StorePaths outPaths;
for (auto & path : toBuiltPaths(evalStore, store, mode, operateOn, installables)) {
auto thisOutPaths = path.outPaths2();
auto thisOutPaths = path.outPaths();
outPaths.insert(outPaths.end(), thisOutPaths.begin(), thisOutPaths.end());
}
return outPaths;
Expand Down

0 comments on commit 858cb92

Please sign in to comment.