On 17/10/12 15:04, Alex Guerra wrote:
Signed-off-by: Alex Guerra<alex.gue...@oiwifi.com.br>
---
  .../network/services/dnsmasq/files/dnsmasq.init    |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/network/services/dnsmasq/files/dnsmasq.init
b/package/network/services/dnsmasq/files/dnsmasq.init
index 85c22c3..2f81cef 100644
--- a/package/network/services/dnsmasq/files/dnsmasq.init
+++ b/package/network/services/dnsmasq/files/dnsmasq.init
@@ -131,7 +131,7 @@ dnsmasq() {
         config_get_bool cachelocal "$cfg" cachelocal 1

         config_get hostsfile "$cfg" dhcphostsfile
-       [ -e "$hostsfile" ]&&  xappend "--dhcp-hostsfile=$hostsfile"
+       [ -n "$hostsfile" ]&&  xappend "--dhcp-hostsfile=$hostsfile"

         local rebind
         config_get_bool rebind "$cfg" rebind_protection 1


the subject should be shorter and the currect subject should be moved to the description. also the description which is currently the subject is vague and not really consisent with what the patch does. the check is not removed but replced with a check that tests if the $hostsfile is set

_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to