From owner-freebsd-hackers@FreeBSD.ORG Thu Jan 6 13:45:31 2011 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 1233) id F115C1065672; Thu, 6 Jan 2011 13:45:31 +0000 (UTC) Date: Thu, 6 Jan 2011 13:45:31 +0000 From: Alexander Best To: Sergey Kandaurov Message-ID: <20110106134531.GA81523@freebsd.org> References: <20110106014046.GA324@freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Cc: freebsd-hackers@freebsd.org Subject: Re: broken INCLUDE in sys/conf/kern.pre.mk for opensolaris code? X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 06 Jan 2011 13:45:32 -0000 On Thu Jan 6 11, Sergey Kandaurov wrote: > On 6 January 2011 04:40, Alexander Best wrote: > > hi there, > > > > while building target buildkernel with 'clang -v' i noticed a lot of these > > lines: > > > > ignoring nonexistent directory "/usr/subversion-src/sys/contrib/opensolaris/compat" > > > > i checked sys/conf/kern.pre.mk and there's a line refering to a non-existing > > directory: > > > > # ...  and OpenSolaris > > INCLUDES+= -I$S/contrib/opensolaris/compat > > I guess that's leftover from early dtrace stages in perforce. > See //depot/projects/dtrace/src/sys/contrib/opensolaris/compat/sys ah i see. whom do i have to ping to get this fixed? i'm not sure if the include is even necessary, because obviosly a kernel will build without it. the only issue i had with cddl code is that when you compile and install world with WITHOUT_CDDL=true, you cannot rebuild world with cddl support. but i don't think that's something the include (even if it points to the correct directory) can fix. cheers. alex > > > > > is suspect this should actually be: > > > > # ...  and OpenSolaris > > INCLUDES+= -I$S/cddl/compat/opensolaris > > > > but i'm not sure of it. > > > > > -- > wbr, > pluknet -- a13x