Date: Thu, 24 Jan 2008 03:02:50 -0500 From: Tom Rhodes <trhodes@FreeBSD.org> To: Matthew May <mdmay74@internode.on.net> 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 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>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 24 Jan 2008 17:52:23 +1030 Matthew May <mdmay74@internode.on.net> 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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080124030250.09cebcd8.trhodes>