From owner-svn-src-head@FreeBSD.ORG  Sat Oct 27 17:26:10 2012
Return-Path: <owner-svn-src-head@FreeBSD.ORG>
Delivered-To: svn-src-head@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
 by hub.freebsd.org (Postfix) with ESMTP id BC927D4F;
 Sat, 27 Oct 2012 17:26:10 +0000 (UTC)
 (envelope-from utisoft@gmail.com)
Received: from mail-bk0-f54.google.com (mail-bk0-f54.google.com
 [209.85.214.54])
 by mx1.freebsd.org (Postfix) with ESMTP id B57E68FC0A;
 Sat, 27 Oct 2012 17:26:09 +0000 (UTC)
Received: by mail-bk0-f54.google.com with SMTP id jf20so1622173bkc.13
 for <multiple recipients>; Sat, 27 Oct 2012 10:26:08 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;
 h=mime-version:sender:in-reply-to:references:from:date
 :x-google-sender-auth:message-id:subject:to:cc:content-type;
 bh=FZLBnMCUlcgo6D6JHEiUhPqSK3AI5ypUkVnYvn6XAEo=;
 b=UbAgU7EQvBsVUjj0AIpoaJ7LXTLMhWSJfP0X236PokpqykkM+tTWQAShXn8ziXOK6E
 UVcMVI9Kgl8Q4EY0M9nqF4reWpYcI4uKbcg1SlLf0paCyOX6r35xLz2P+n2o0nzS/4qi
 liDq24edq4GPn6FX/EXp3M8HT9LbLz5pMwaqlaV3SllWRjiTqwtcXo75y9B/cS0nZkmE
 yyPQTIMfKFq4uN3TYkaSIGY/XiOVJwGMZucvTouDC8hUKdktRzZ+xoDsqFhv6F5/67Or
 iGYdibmdpZgD6+hIJ5b0GwVx87G9v7a5fj4wmwP/c1CVxy45ovCDO5lsy18fnzGDiHjk
 lvQg==
Received: by 10.204.150.213 with SMTP id z21mr8231493bkv.45.1351358768457;
 Sat, 27 Oct 2012 10:26:08 -0700 (PDT)
MIME-Version: 1.0
Sender: utisoft@gmail.com
Received: by 10.204.50.197 with HTTP; Sat, 27 Oct 2012 10:25:38 -0700 (PDT)
In-Reply-To: <20121028.022034.1805071030218844630.hrs@allbsd.org>
References: <201210271706.q9RH6Qu4018997@svn.freebsd.org>
 <CADLo83-psYE3T9+aZhEfQWpm6Ht3snC8OrQ=1Qarhnv124OQqg@mail.gmail.com>
 <20121028.022034.1805071030218844630.hrs@allbsd.org>
From: Chris Rees <crees@FreeBSD.org>
Date: Sat, 27 Oct 2012 18:25:38 +0100
X-Google-Sender-Auth: C5pwmrEWolFcp7Ip2J7Q9bcO6_Q
Message-ID: <CADLo83-74RG_3OZJrWZK+6a2nvz-qR+q7BoUupMcVEnFYefywA@mail.gmail.com>
Subject: Re: svn commit: r242181 - in head/etc: . rc.d
To: Hiroki Sato <hrs@freebsd.org>
Content-Type: text/plain; charset=ISO-8859-1
Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org,
 src-committers@freebsd.org
X-BeenThere: svn-src-head@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
List-Id: SVN commit messages for the src tree for head/-current
 <svn-src-head.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-head>
List-Post: <mailto:svn-src-head@freebsd.org>
List-Help: <mailto:svn-src-head-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sat, 27 Oct 2012 17:26:10 -0000

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