From owner-freebsd-doc@freebsd.org Wed Aug 24 03:41:12 2016 Return-Path: Delivered-To: freebsd-doc@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 240E2BC2AB8 for ; Wed, 24 Aug 2016 03:41:12 +0000 (UTC) (envelope-from cy.schubert@komquats.com) Received: from smtp-out-no.shaw.ca (smtp-out-no.shaw.ca [64.59.134.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "Client", Issuer "CA" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id D3CDE16F5; Wed, 24 Aug 2016 03:41:11 +0000 (UTC) (envelope-from cy.schubert@komquats.com) Received: from spqr.komquats.com ([96.50.22.10]) by shaw.ca with SMTP id cP3lbm5ZAnROdcP3nbaXlo; Tue, 23 Aug 2016 21:41:05 -0600 X-Authority-Analysis: v=2.2 cv=eo2d9chX c=1 sm=1 tr=0 a=jvE2nwUzI0ECrNeyr98KWA==:117 a=jvE2nwUzI0ECrNeyr98KWA==:17 a=L9H7d07YOLsA:10 a=9cW_t1CCXrUA:10 a=s5jvgZ67dGcA:10 a=7z1cN_iqozsA:10 a=NEAV23lmAAAA:8 a=85N1-lAfAAAA:8 a=YxBL1-UpAAAA:8 a=6I5d2MoRAAAA:8 a=VKzZUOvROZqElCnfFvwA:9 a=CjuIK1q_8ugA:10 a=UJ0tAi3fqDAA:10 a=Bn2pgwyD2vrAyMmN8A2t:22 a=cyfSibbquD4hpIoiQNSb:22 a=Ia-lj3WSrqcvXOmTRaiG:22 a=IjZwj45LgO3ly-622nXo:22 Received: from slippy.cwsent.com (slippy8 [10.2.2.6]) by spqr.komquats.com (Postfix) with ESMTPS id AAF7813754; Tue, 23 Aug 2016 20:41:01 -0700 (PDT) Received: from slippy (localhost [127.0.0.1]) by slippy.cwsent.com (8.15.2/8.15.2) with ESMTP id u7O3ex9c004738; Tue, 23 Aug 2016 20:41:00 -0700 (PDT) (envelope-from Cy.Schubert@cschubert.com) Message-Id: <201608240341.u7O3ex9c004738@slippy.cwsent.com> X-Mailer: exmh version 2.8.0 04/21/2012 with nmh-1.6 Reply-to: Cy Schubert From: Cy Schubert X-os: FreeBSD X-Sender: cy@cwsent.com X-URL: http://www.cschubert.com/ To: Benjamin Kaduk cc: Steve Kargl , cy@freebsd.org, freebsd-doc@freebsd.org, delphij@freebsd.org, Ollivier Robert Subject: Re: [PATCH] Fix markup in sntp.8 In-Reply-To: Message from Benjamin Kaduk of "Tue, 23 Aug 2016 21:32:57 -0400." Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Tue, 23 Aug 2016 20:40:59 -0700 X-CMAE-Envelope: MS4wfK/lYgVufEiOZFxxFfiERck0eSQeB4PSSaudXqGw03sqvpubWpxkzNKHURtxvrhPfE2680O8GTD+spNsYwLGqVz93XHzRKlHO/kelNg63K7v3K6Wv1ae vX7OqZm0W/05Efn2CUnKu75ObqPEO8W1eAeT/cdssHkjAMRCevXqWd1yoiSp+RQW0VI0w7UxJgnoJwSeu3KU4rKTzmixGLYVon9sZAn209AQrda/4PZ577BS aM6V6D74ZH4arMLB7ms+7mpG7hAcLaoWbLsPjaMoBYHpKKhq6kGiqtHasr1wR8P49fNNa2HukA3FlcP6Dtnt4gwlh3ziOmXW8gmFSIV+ktmFSCloi/gBIGfy rAmY2UkX X-BeenThere: freebsd-doc@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: Documentation project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 24 Aug 2016 03:41:12 -0000 Let's try this again... I had my cursor in the wrong place in exmh when hit send, making my reply look a little dumb. In message , Benjamin Kaduk writes: > On Tue, 23 Aug 2016, Benjamin Kaduk wrote: > > > On Tue, 23 Aug 2016, Steve Kargl wrote: > > > > > On Wed, Aug 17, 2016 at 01:47:37PM -0700, Steve Kargl wrote: > > > > Index: sntp.8 > > > > =================================================================== > > > > --- sntp.8 (revision 303492) > > > > +++ sntp.8 (working copy) > > > > @@ -213,7 +213,7 @@ > > > > more than enough for a unicast response. If \fBsntp\fP is > > > > only waiting for a broadcast response a longer timeout is > > > > likely needed. > > > > -.It Fl \-wait , " Fl \-no\-wait" > > > > +.It Fl \-wait , Fl \-no\-wait > > > > Wait for pending replies (if not setting the time). > > > > The \fIno\-wait\fP form will disable the option. > > > > This option is enabled by default. > > > > > > > > > > Does freebsd-doc handle documentation patches? > > > > We do handle them, we are just sometimes slow about it. Hi Ben, Either the doc team or src team can handle this. I added Ollivier Robert, FreeBSD's original src/ NTP mainainer, to the cc list. > > Getting around towards doing so, I note that this file is from a vendor > tree, and the bug is still present in > https://github.com/ntp-project/ntp/blob/master/sntp/sntp.mdoc.in . > Unfortunately, http://bugs.ntp.org/enter_bug.cgi wants me to create a > login; adding cy and delphij to see if either of them already have one and > would be able to report the bug upstream. I have an account. I can submit a bug upstream. As port maintainer (as well) I've communicated with our upstream (Harlan Stenn) on many an occasion. > > My understanding is that now that we're in subversion intsead of CVS, > applying a local patch to a file from the vendor tree is not a huge > burden, but I'm inclined to wait a bit to add a local patch so we can see > how responsive upstream is. This gets a little complicated. Even though the man pages, like sntp.man.in, are in the vendor branch and in contrib, they're actually installed from usr.sbin/ntp/doc. It's not "right" and it should be. Using the vendor supplied man pages is preferred and is on my list of things to do. Having said that, as ntp ports (ntp and ntp-devel) maintainer, I've seen a fair bit of fluctuation with ntp documentation where sometimes there are man pages and at other times the man pages are replaced by html documents. The issue with this is that it will create a bit of churn for ntp in src/ because sometime man pages will be available and sometimes not. The solution (I think) is to use vendor supplied man pages when available and when not, either not use the man pages in usr.sbin/ntp/doc or svn cp missing man pages back. To summarize, this needs to be thought out a little more in regard to revering to previous man pages when those in the distribution are deleted in lieu of html documentation (yet again). But I digress, this is getting off topic. I've commited the patch to base and ports, have submitted the patch upstream, and have emailed Harlan Stenn (personally) as well. > > Sorry about the (further) delay... No problem, just not enough time in the day any more. Thanks for email Ben. -- Cheers, Cy Schubert FreeBSD UNIX: Web: http://www.FreeBSD.org The need of the many outweighs the greed of the few.