maurice/snix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix "nix ... --all"

Browse source 
When "--all" is used, we should not fill in a default installable.
This commit is contained in:
Eelco Dolstra 2017-05-02 15:28:35 +02:00
parent 7dedd3fa24
commit cef8c169b1
No known key found for this signature in database
GPG key ID: 8170B4726D7198DE
2 changed files with 16 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/nix/command.hh
View file

@ -78,7 +78,7 @@ struct InstallablesCommand : virtual Args, StoreCommand
= import ...; bla = import ...; }’. */
Value * getSourceExpr(EvalState & state);
std::vector<std::shared_ptr<Installable>> parseInstallables(ref<Store> store, Strings installables);
std::vector<std::shared_ptr<Installable>> parseInstallables(ref<Store> store, Strings ss);
PathSet buildInstallables(ref<Store> store, bool dryRun);
@ -86,6 +86,8 @@ struct InstallablesCommand : virtual Args, StoreCommand
void prepare() override;
virtual bool useDefaultInstallables() { return true; }
private:
Strings _installables;
@ -112,6 +114,8 @@ public:
virtual void run(ref<Store> store, Paths storePaths) = 0;
void run(ref<Store> store) override;
bool useDefaultInstallables() override { return !all; }
};
typedef std::map<std::string, ref<Command>> Commands;

22
src/nix/installables.cc
View file

@ -177,21 +177,21 @@ struct InstallableAttrPath : Installable
std::string attrRegex = R"([A-Za-z_][A-Za-z0-9-_+]*)";
static std::regex attrPathRegex(fmt(R"(%1%(\.%1%)*)", attrRegex));
std::vector<std::shared_ptr<Installable>> InstallablesCommand::parseInstallables(ref<Store> store, Strings installables)
std::vector<std::shared_ptr<Installable>> InstallablesCommand::parseInstallables(ref<Store> store, Strings ss)
{
std::vector<std::shared_ptr<Installable>> result;
if (installables.empty()) {
if (ss.empty() && useDefaultInstallables()) {
if (file == "")
file = ".";
installables = Strings{""};
ss = Strings{""};
}
for (auto & installable : installables) {
for (auto & s : ss) {
if (installable.find("/") != std::string::npos) {
if (s.find("/") != std::string::npos) {
auto path = store->toStorePath(store->followLinksToStore(installable));
auto path = store->toStorePath(store->followLinksToStore(s));
if (store->isStorePath(path)) {
if (isDerivation(path))
@ -201,14 +201,14 @@ std::vector<std::shared_ptr<Installable>> InstallablesCommand::parseInstallables
}
}
else if (installable.compare(0, 1, "(") == 0)
result.push_back(std::make_shared<InstallableExpr>(*this, installable));
else if (s.compare(0, 1, "(") == 0)
result.push_back(std::make_shared<InstallableExpr>(*this, s));
else if (installable == "" || std::regex_match(installable, attrPathRegex))
result.push_back(std::make_shared<InstallableAttrPath>(*this, installable));
else if (s == "" || std::regex_match(s, attrPathRegex))
result.push_back(std::make_shared<InstallableAttrPath>(*this, s));
else
throw UsageError("don't know what to do with argument ‘%s’", installable);
throw UsageError("don't know what to do with argument ‘%s’", s);
}
return result;