From owner-freebsd-doc@FreeBSD.ORG Thu Jan 24 08:02:57 2008 Return-Path: Delivered-To: freebsd-doc@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6B9D016A4CB; Thu, 24 Jan 2008 08:02:57 +0000 (UTC) (envelope-from trhodes@FreeBSD.org) Received: from pittgoth.com (pittgoth.com [205.134.163.206]) by mx1.freebsd.org (Postfix) with ESMTP id 3151F13C448; Thu, 24 Jan 2008 08:02:57 +0000 (UTC) (envelope-from trhodes@FreeBSD.org) Received: from localhost.fbsdsecure.org (net-ix.gw.ai.net [205.134.160.6]) (authenticated bits=0) by pittgoth.com (8.13.6/8.13.6) with ESMTP id m0O82t7Q034060 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Thu, 24 Jan 2008 03:02:56 -0500 (EST) (envelope-from trhodes@FreeBSD.org) Date: Thu, 24 Jan 2008 03:02:50 -0500 From: Tom Rhodes To: Matthew May Message-Id: <20080124030250.09cebcd8.trhodes@FreeBSD.org> In-Reply-To: <47983CAF.8010509@internode.on.net> References: <200801240150.m0O1o2CW086551@freefall.freebsd.org> <47983CAF.8010509@internode.on.net> X-Mailer: Sylpheed version 1.0.6 (GTK+ 1.2.10; i386-portbld-freebsd7.0) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: trhodes@FreeBSD.org, freebsd-doc@FreeBSD.org Subject: Re: docs/118261: [patch] Base NTP: NO_NTP support + NO_SHAREDOCS applies to NTP's HTMLs X-BeenThere: freebsd-doc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Documentation project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Jan 2008 08:02:57 -0000 On Thu, 24 Jan 2008 17:52:23 +1030 Matthew May wrote: > > Tom Rhodes wrote: > ... > > wraps the share docs into NO_SHAREDOCS. The only issues I > > think the only other changes to this patch would probably be > > to not enter the docs directory at all if both NO_SHAREDOCS > > and NO_MAN are set. This could be done with a different change > > to the Makefile if: > > > > .if !defined(NO_SHAREDOCS) && !defined(NO_MAN) > > subdir+= doc > > .endif > > I may be wrong, but wouldn't that be: > .if !defined(NO_SHAREDOCS) || !defined(NO_MAN) > ? :-) No, you're right. In fact, my patch is actually wrong. The NO_HTML option exists on RELENG_6 so I'm going to use that. I'll also remove NO_MAN from the Makefile as it's handled by bsd.prog.mk. :) -- Tom Rhodes