[oe] [meta-networking][PATCH 2/2] dnsmasq: disable the service by default

[Bruce Ashfield]

On Mon, Feb 27, 2017 at 3:09 AM, <jackie.huang at windriver.com> wrote:

> From: Jackie Huang <jackie.huang at windriver.com>
>
> The service will fail to start/stop when dnsmasq and
> bind (provides named) are both installed since they
> are using the same port 53:
>
> | dnsmasq: failed to create listening socket for port 53: Address already
> in use
>
> So disable the service by default, if the user really
> want to use two dns servives, the config needs to be
> changed to use different interfaces or ports.
>


But what if the user is only using dnsmasq ? (which is the case in the
images I use).
This would be a behaviour change that they add dnsmasq, but don't actually
get the
service started.

IMHO if someone is installing two packages that provide the same service,
they
are more aware of what is going on and can disable one (or the other) ..
versus
someone that adds a single package that provides a service .. and then find
out that
it doesn't actually start.

Bruce


>
> As reference, other distro like centos/fedora also
> disables the services by default on installation.
>
> Signed-off-by: Jackie Huang <jackie.huang at windriver.com>
> ---
>  meta-networking/recipes-support/dnsmasq/dnsmasq.inc | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta-networking/recipes-support/dnsmasq/dnsmasq.inc
> b/meta-networking/recipes-support/dnsmasq/dnsmasq.inc
> index 424a06b..9c2276f 100644
> --- a/meta-networking/recipes-support/dnsmasq/dnsmasq.inc
> +++ b/meta-networking/recipes-support/dnsmasq/dnsmasq.inc
> @@ -81,3 +81,4 @@ RPROVIDES_${PN} += "${PN}-systemd"
>  RREPLACES_${PN} += "${PN}-systemd"
>  RCONFLICTS_${PN} += "${PN}-systemd"
>  SYSTEMD_SERVICE_${PN} = "dnsmasq.service"
> +SYSTEMD_AUTO_ENABLE ?= "disable"
> --
> 2.8.3
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>



-- 
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"