Remove safe defaults for dnsdist packages (is included in the binary), fixes the...

diff --git a/build-scripts/build-dnsdist-rpm b/build-scripts/build-dnsdist-rpm
index c01f9179c73239b7561a770c18f42e1f328f7c6a..aded5f533b7190d751a04662f8fb9f02df5f5195 100755 (executable)
--- a/build-scripts/build-dnsdist-rpm
+++ b/build-scripts/build-dnsdist-rpm
@@ -39,9 +39,9 @@ INIT_BUILDREQUIRES='BuildRequires: systemd'
 INIT_INSTALL='install -d -m 755 %{buildroot}/%{_sysconfdir}/systemd/system/ && install -m 664 contrib/dnsdist.service %{buildroot}/%{_sysconfdir}/systemd/system/dnsdist.service'
 INIT_FILES='%{_sysconfdir}/systemd/system/dnsdist.service'
 
-# These two are the same for sysv and systemd
-DEFAULTS_INSTALL='install -d -m 755 %{buildroot}/%{_sysconfdir}/sysconfig && install -m 644 contrib/dnsdist.default %{buildroot}/%{_sysconfdir}/sysconfig/dnsdist'
-DEFAULTS_FILES='%{_sysconfdir}/sysconfig/dnsdist'
+# These two are the same for sysv and systemd (we don't install defaults files at the moment)
+DEFAULTS_INSTALL=''
+DEFAULTS_FILES=''
 
 # On some older distro's *cough* centos 6 *cough* autosetup fails
 SETUP='%autosetup -n %{name}-%{version}'

diff --git a/build-scripts/debian-dnsdist/dnsdist.default b/build-scripts/debian-dnsdist/dnsdist.default
deleted file mode 100644 (file)
index 622ed68..0000000
--- a/build-scripts/debian-dnsdist/dnsdist.default
+++ /dev/null
@@ -1 +0,0 @@
-OPTIONS='-l 127.0.0.1'

diff --git a/build-scripts/debian-dnsdist/dnsdist.service b/build-scripts/debian-dnsdist/dnsdist.service
index 40004acd8d58963b30ecacad812d736900138944..e0374e843c2802ca606c3a16915e92b5ebbd041e 100644 (file)
--- a/build-scripts/debian-dnsdist/dnsdist.service
+++ b/build-scripts/debian-dnsdist/dnsdist.service
@@ -3,8 +3,8 @@ Description=DNSdist
 After=syslog.target
 
 [Service]
-EnvironmentFile=-/etc/default/dnsdist
-ExecStart=/usr/bin/dnsdist --supervised ${OPTIONS}
+# Keep the --supervised option when modifying the default options
+ExecStart=/usr/bin/dnsdist --supervised
 
 [Install]
 WantedBy=multi-user.target

diff --git a/build-scripts/debian-dnsdist/dnsdist.upstart b/build-scripts/debian-dnsdist/dnsdist.upstart
index 191dab2d9e9acc0f5fbe1e7d08152f0610710fbb..0ded50f9ab86dfd858b2d134482e9e2572e915d6 100644 (file)
--- a/build-scripts/debian-dnsdist/dnsdist.upstart
+++ b/build-scripts/debian-dnsdist/dnsdist.upstart
@@ -6,4 +6,4 @@ stop on runlevel [!2345]
 author "Pieter Lexis <pieter.lexis@powerdns.com>"
 
 # Keep the --supervised option when modifying this
-exec /usr/bin/dnsdist --supervised -l 127.0.0.1
+exec /usr/bin/dnsdist --supervised

diff --git a/pdns/dnsdistdist/contrib/dnsdist.default b/pdns/dnsdistdist/contrib/dnsdist.default
index 4e0e04529f5364ddd922dd90e6fe5551da761775..8e6dc5b9a72a8f77b4af039472cd5d8a62f64601 100644 (file)
--- a/pdns/dnsdistdist/contrib/dnsdist.default
+++ b/pdns/dnsdistdist/contrib/dnsdist.default
@@ -1 +1 @@
-DNSDIST_OPTIONS='-l 127.0.0.1'
+DNSDIST_OPTIONS=''

diff --git a/pdns/dnsdistdist/contrib/dnsdist.init.centos6 b/pdns/dnsdistdist/contrib/dnsdist.init.centos6
index 613481c88c83e8e517b2de4d8c06e1ac7600eaf3..071fa0b35aadc0d6550bd3232fd1fa923b46b670 100644 (file)
--- a/pdns/dnsdistdist/contrib/dnsdist.init.centos6
+++ b/pdns/dnsdistdist/contrib/dnsdist.init.centos6
@@ -17,12 +17,16 @@
 PROG=dnsdist
 DNSDIST=/usr/bin/${PROG}
 PIDFILE=/var/run/${PROG}.pid
-DNSDIST_OPTIONS="-l 127.0.0.1:53"
+DNSDIST_OPTIONS=""
 
 if [ -f /etc/default/${PROG} ]; then
   . /etc/default/${PROG}
 fi
 
+if [ -f /etc/sysconfig/${PROG} ]; then
+  . /etc/sysconfig/${PROG}
+fi
+
 RETVAL=0
 
 do_start() {

diff --git a/pdns/dnsdistdist/contrib/dnsdist.service b/pdns/dnsdistdist/contrib/dnsdist.service
index 00a7bfda438541e154d64ddde05a10cea3caa6ba..11067dfc8a692fcad057d49dcbf208c8629a37f1 100644 (file)
--- a/pdns/dnsdistdist/contrib/dnsdist.service
+++ b/pdns/dnsdistdist/contrib/dnsdist.service
@@ -3,8 +3,8 @@ Description=dnsdist
 After=network.target
 
 [Service]
-EnvironmentFile=-/etc/sysconfig/dnsdist
-ExecStart=/usr/bin/dnsdist --supervised $DNSDIST_OPTIONS
+# Keep the --supervised option when changing the default options
+ExecStart=/usr/bin/dnsdist --supervised
 
 [Install]
 WantedBy=multi-user.target