Date: Sat, 25 Mar 2006 00:35:02 +1100 (EST) From: Bruce Evans <bde@zeta.org.au> To: Mike Silbersack <silby@silby.com> Cc: src-committers@FreeBSD.org, Christian Brueffer <chris@unixpages.org>, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org, Marcel Moolenaar <marcel@xcllnt.net>, Kris Kennaway <kris@obsecurity.org> Subject: Re: cvs commit: src/share/man/man4 ddb.4 Message-ID: <20060325003135.I7209@epsplex.bde.org> In-Reply-To: <20060324021513.C94486@odysseus.silby.com> References: <200602272156.k1RLuOnF050808@repoman.freebsd.org> <20060228011204.GA56781@ns1.xcllnt.net> <20060228011935.GA26407@xor.obsecurity.org> <20060228013742.GA56890@ns1.xcllnt.net> <20060228063546.GA1898@haakonia.hitnet.RWTH-Aachen.DE> <4C55DEF0-E63A-4305-995F-3191B99011DA@xcllnt.net> <20060324021513.C94486@odysseus.silby.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 24 Mar 2006, Mike Silbersack wrote: > On Tue, 28 Feb 2006, Marcel Moolenaar wrote: > >> There's a bogus conditional in i386/i386/machdep.c that should be >> removed: >> >> revision 1.596 >> date: 2004/07/19 02:46:34; author: silby; state: Exp; lines: +3 -0 >> Add a #error requiring KDB if DDB is specified. (This can probably be >> relocated to a better place, if one exists.) >> >> It has found its way to amd64 as well and therefore may be elsewhere >> (not ia64, as that works as expected). >> >> -- >> Marcel Moolenaar USPA: A-39004 marcel@xcllnt.net > > I added that conditional because DDB w/o KDB didn't work at the time, and the > weird way it failed to link was very confusing. If the two are now > independent, then by all means remove the check. Removing it doesn't work in at least 6.0. IIRC, the error involves KDB not knowing that DDB is there. Bruce
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060325003135.I7209>