Date: Sat, 27 Oct 2012 18:25:38 +0100 From: Chris Rees <crees@FreeBSD.org> To: Hiroki Sato <hrs@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r242181 - in head/etc: . rc.d Message-ID: <CADLo83-74RG_3OZJrWZK%2B6a2nvz-qR%2Bq7BoUupMcVEnFYefywA@mail.gmail.com> In-Reply-To: <20121028.022034.1805071030218844630.hrs@allbsd.org> References: <201210271706.q9RH6Qu4018997@svn.freebsd.org> <CADLo83-psYE3T9%2BaZhEfQWpm6Ht3snC8OrQ=1Qarhnv124OQqg@mail.gmail.com> <20121028.022034.1805071030218844630.hrs@allbsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 27 October 2012 18:20, Hiroki Sato <hrs@freebsd.org> wrote: > Chris Rees <crees@FreeBSD.org> wrote > in <CADLo83-psYE3T9+aZhEfQWpm6Ht3snC8OrQ=1Qarhnv124OQqg@mail.gmail.com>: > > cr> On 27 October 2012 18:06, Hiroki Sato <hrs@freebsd.org> wrote: > cr> > Author: hrs > cr> > Date: Sat Oct 27 17:06:26 2012 > cr> > New Revision: 242181 > cr> > URL: http://svn.freebsd.org/changeset/base/242181 > cr> > > cr> > Log: > cr> > Fix an issue when ipv6_enable=YES && ipv6_gateway_enable=YES which could > cr> > prevent rtadvd(8) from working as intended. > cr> > > cr> > Spotted by: brian > cr> > Discussed with: brian > cr> > > cr> > Modified: > cr> > head/etc/network.subr > cr> > head/etc/rc.d/rtadvd > cr> > > cr> > Modified: head/etc/network.subr > cr> > ============================================================================== > cr> > --- head/etc/network.subr Sat Oct 27 16:44:41 2012 (r242180) > cr> > +++ head/etc/network.subr Sat Oct 27 17:06:26 2012 (r242181) > cr> > @@ -504,7 +504,11 @@ ipv6_autoconfif() > cr> > # backward compatibility: $ipv6_enable > cr> > case $ipv6_enable in > cr> > [Yy][Ee][Ss]|[Tt][Rr][Uu][Ee]|[Oo][Nn]|1) > cr> > - return 0 > cr> > + if checkyesno ipv6_gateway_enable; then > cr> > + return 1 > cr> > + else > cr> > + return 0 > cr> > + fi > cr> > cr> While there, any chance to use checkyesno ipv6_enable too? > > No, this is intentional because ipv6_enable is a deprecated variable > and a "not set properly" warning should not be displayed even if it > is not defined. Thanks for clarifying. Chris
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADLo83-74RG_3OZJrWZK%2B6a2nvz-qR%2Bq7BoUupMcVEnFYefywA>