From: bert hubert <bert.hubert@netherlabs.nl>
Date: Fri, 24 May 2013 13:34:52 +0000 (+0200)
Subject: Fixes #571 "by error message". The problem is that we can't detect the difference... 
X-Git-Tag: auth-3.3-rc1~10
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b925384e96597b0c3135a7833ba1d9e853d5ce95;p=pdns

Fixes #571 "by error message". The problem is that we can't detect the 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.
---

diff --git a/pdns/pdnssec.cc b/pdns/pdnssec.cc
index 78cac42f5..d218bbf2c 100644
--- a/pdns/pdnssec.cc
+++ b/pdns/pdnssec.cc
@@ -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;
   }
 
@@ -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;