Skip to content

Commit

Permalink
Fixes #571 "by error message". The problem is that we can't detect th…
Browse files Browse the repository at this point in the history
…e difference between not setting a configuration item to the empty string and not including it at all.

For switches we get away with that by having an explicit non-empty default, but we can't do that for filenames.
  • Loading branch information
ahupowerdns committed May 24, 2013
1 parent 07bf35d commit b925384
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions pdns/pdnssec.cc
Original file line number Diff line number Diff line change
Expand Up @@ -773,7 +773,8 @@ bool secureZone(DNSSECKeeper& dk, const std::string& zone)
cerr<<"Failed to secure zone. Is your backend dnssec enabled? (set \n";
cerr<<"gsqlite3-dnssec, or gmysql-dnssec etc). Check this first.\n";
cerr<<"If you run with the BIND backend, make sure you have configured\n";
cerr<<"it to use DNSSEC with 'bind-dnssec-db' and 'pdnssec create-bind-db'!\n";
cerr<<"it to use DNSSEC with 'bind-dnssec-db=/path/fname' and\n";
cerr<<"'pdnssec create-bind-db /path/fname'!\n";
return false;
}

Expand Down Expand Up @@ -974,8 +975,7 @@ try

loadMainConfig(g_vm["config-dir"].as<string>());
reportAllTypes();



if(cmds[0] == "create-bind-db") {
if(cmds.size() != 2) {
cerr << "Syntax: pdnssec create-bind-db fname"<<endl;
Expand Down

0 comments on commit b925384

Please sign in to comment.