From owner-freebsd-rc@FreeBSD.ORG Fri Dec 23 17:17:18 2011 Return-Path: Delivered-To: freebsd-rc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 378C21065689 for ; Fri, 23 Dec 2011 17:17:18 +0000 (UTC) (envelope-from Devin.Teske@fisglobal.com) Received: from mx1.fisglobal.com (mx1.fisglobal.com [199.200.24.190]) by mx1.freebsd.org (Postfix) with ESMTP id EFD318FC24 for ; Fri, 23 Dec 2011 17:17:17 +0000 (UTC) Received: from pps.filterd (ltcfislmsgpa06 [127.0.0.1]) by ltcfislmsgpa06.fnfis.com (8.14.4/8.14.4) with SMTP id pBNGT94p016760 for ; Fri, 23 Dec 2011 11:17:16 -0600 Received: from smtp.fisglobal.com ([10.132.206.17]) by ltcfislmsgpa06.fnfis.com with ESMTP id 11vvtk89hp-5 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NOT) for ; Fri, 23 Dec 2011 11:17:16 -0600 Received: from [10.0.0.103] (10.14.152.38) by smtp.fisglobal.com (10.132.206.17) with Microsoft SMTP Server (TLS) id 14.1.323.3; Fri, 23 Dec 2011 11:17:15 -0600 From: Devin Teske Content-Type: multipart/mixed; boundary="Apple-Mail-38-608566932" Date: Fri, 23 Dec 2011 09:17:14 -0800 Message-ID: To: FreeBSD RC MIME-Version: 1.0 (Apple Message framework v1084) X-Mailer: Apple Mail (2.1084) X-Originating-IP: [10.14.152.38] X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.5.7110, 1.0.211, 0.0.0000 definitions=2011-12-23_05:2011-12-23, 2011-12-23, 1970-01-01 signatures=0 Cc: Dave Robison Subject: rc.d/named default pidfile X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Dec 2011 17:17:18 -0000 --Apple-Mail-38-608566932 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="us-ascii" Working on named(8) with a co-worker yesterday, we were surprised to see wh= at appears to be a typo in the rc.d script in FreeBSD-8.1. Please find attached a patch for HEAD (not FreeBSD-8.1; the typo still exis= ts in HEAD). _____________ The information contained in this message is proprietary and/or confidentia= l. If you are not the intended recipient, please: (i) delete the message an= d all copies; (ii) do not disclose, distribute or use the message in any ma= nner; and (iii) notify the sender immediately. In addition, please be aware= that any message addressed to our domain is subject to archiving and revie= w by persons other than the intended recipient. Thank you. --Apple-Mail-38-608566932 Content-Disposition: attachment; filename="named.patch" Content-Type: application/octet-stream; name="named.patch" Content-Transfer-Encoding: 7bit --- HEAD/src/etc/rc.d/named.orig Sun May 15 20:25:41 2011 +++ HEAD/src/etc/rc.d/named Fri Dec 23 08:50:17 2011 @@ -117,7 +117,7 @@ find_pidfile() if get_pidfile_from_conf pid-file $named_conf; then pidfile="$_pidfile_from_conf" else - pidfile="/var/run/named/pid" + pidfile="/var/run/named.pid" fi } --Apple-Mail-38-608566932 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="us-ascii" Details/History: In FreeBSD-8.1, the rc.d script declares named(8) isn't running when = it-is because the default pidfile is incorrectly "/var/run/named/pid" = when the program itself writes "/var/run/named.pid" (notice the trailing = ".pid" versus "/pid"). In SVN rev 220962 the rc.d script was changed by dougb to dynamically = determine the pidfile location from the configuration file. When this = patch was applied, the rc.d script started working again w/respect to = "status" (accurately reported when named(8) was/wasn't running). = Unfortunately, the typo still exists (see patch for HEAD). I would like to request that this patch be made against HEAD, and the = changes be MFC'd to stable/8, ultimately to be ready for the release/8.3 = branch when copied (we're planning on skipping 8.2 and jumping to 8.3 as = our next release; currently being on 8.1). Aside: On FreeBSD-8.1, if someone wants to make the appropriate fix, the change = must be made in two places. src/etc/rc.d/named and = src/etc/defaults/rc.conf Work-around in FreeBSD-8.1 and older (not sure about 8.2, but RELENG_8 = appears to have received SVN rev 220962 so named_pidfile no longer has = any effect): Add named_pidfile=3D"/var/run/named.pid" to your rc.conf(5) --=20 Devin= --Apple-Mail-38-608566932--