On Mon, 18 Feb 2019 at 04:00, Hans Dedecker <[email protected]> wrote:
>
> Hi Yousong,
> On Thu, Feb 14, 2019 at 1:15 PM Yousong Zhou <[email protected]> wrote:Y
> >
> > Option resolvfile should now affect only how dnsmasq itself will run.
> > The implicit effect of its specific value on other parts of the system
> > makes the code unnecessarily hard to follow
> >
> > Signed-off-by: Yousong Zhou <[email protected]>
> > ---
> > package/network/services/dnsmasq/Makefile | 2 +-
> > package/network/services/dnsmasq/files/dnsmasq.init | 7 +------
> > 2 files changed, 2 insertions(+), 7 deletions(-)
> >
> > diff --git a/package/network/services/dnsmasq/Makefile
> > b/package/network/services/dnsmasq/Makefile
> > index d51f23e48b..5e83b4f8e5 100644
> > --- a/package/network/services/dnsmasq/Makefile
> > +++ b/package/network/services/dnsmasq/Makefile
> > @@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
> >
> > PKG_NAME:=dnsmasq
> > PKG_VERSION:=2.80
> > -PKG_RELEASE:=8
> > +PKG_RELEASE:=9
> >
> > PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.xz
> > PKG_SOURCE_URL:=http://thekelleys.org.uk/dnsmasq
> > diff --git a/package/network/services/dnsmasq/files/dnsmasq.init
> > b/package/network/services/dnsmasq/files/dnsmasq.init
> > index 05c555bd8c..22471c11aa 100644
> > --- a/package/network/services/dnsmasq/files/dnsmasq.init
> > +++ b/package/network/services/dnsmasq/files/dnsmasq.init
> > @@ -888,7 +888,6 @@ dnsmasq_start()
> > config_get resolvfile "$cfg" resolvfile
> > [ -n "$resolvfile" -a ! -e "$resolvfile" ] && touch
> > "$resolvfile"
> > xappend "--resolv-file=$resolvfile"
> > - [ "$resolvfile" = "/tmp/resolv.conf.auto" ] && localuse=1
> By removing this line 127.0.0.1 is not added anymore in
> /tmp/resolv.conf as localuse is 0 by default;
> as a result local resolving is broken.
Yes, this is exactly the behavior change. It intends to make it clear
that to use dnsmasq for local dns query, one has to set explicitly
localuse to 1. And with localuse being 0, /etc/resolv.conf will
default to /tmp/resolv.conf.auto as setup by /etc/init.d/boot
The argument is mainly that resolvfile being /tmp/resolv.conf.auto
should not have a consequence on the content of /tmp/resolv.conf.
resolvfile is dnsmasq specific and /tmp/resolv.conf is for all other
processes in the local system
Should have made it more clear in the commit message that this can
break existing expectations. Sorry about that, I will drop this
change, amend the first, then send another version.
Thank you,
yousong
_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel
