Re: [LEDE-DEV] [PATCH] dnsmasq: fix error in call of list_contains()

2017-06-13 Thread e9hack
Hi, it seems there is a second bug. [ -n $notinterfaces ] fails, if notinterfaces contains spaces. Regards, Hartmut Am 12.06.2017 um 23:11 schrieb Paul Oranje: > --- a/package/network/services/dnsmasq/files/dnsmasq.init > +++ b/package/network/services/dnsmasq/files/dnsmasq.init > @@ -716,7 +71

[LEDE-DEV] [PATCH] dnsmasq: fix error in call of list_contains()

2017-06-13 Thread Paul Oranje
Fixes the first parameter to the call of list_contains() in dnsmasq_ismain() Signed-off-by: Paul Oranje --- Commit a53f8ba6771de64c9c82a2e6867791226f3003cb introduces an error that has not shown up in my tests of that commit (testing sh script is tough). Thanks to Hartmut (e9hack). --- package/n

Re: [LEDE-DEV] [PATCH] dnsmasq: fix error in call of list_contains()

2017-06-13 Thread Hans Dedecker
On Mon, Jun 12, 2017 at 11:11 PM, Paul Oranje wrote: > Fixes the first parameter to the call of list_contains() in dnsmasq_ismain() > > Signed-off-by: Paul Oranje > --- > Commit a53f8ba6771de64c9c82a2e6867791226f3003cb introduces an error that has > not > shown up in my tests of that commit (tes

[LEDE-DEV] [PATCH] dnsmasq: fix error in call of list_contains()

2017-06-12 Thread Paul Oranje
Fixes the first parameter to the call of list_contains() in dnsmasq_ismain() Signed-off-by: Paul Oranje --- Commit a53f8ba6771de64c9c82a2e6867791226f3003cb introduces an error that has not shown up in my tests of that commit (testing sh script is tough). Thanks to Hartmut (e9hack). --- package/n