From owner-freebsd-current@FreeBSD.ORG Mon Sep 29 12:41:34 2003 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id AC76016A4BF for ; Mon, 29 Sep 2003 12:41:34 -0700 (PDT) Received: from web40302.mail.yahoo.com (web40302.mail.yahoo.com [66.218.78.81]) by mx1.FreeBSD.org (Postfix) with SMTP id D621643FF3 for ; Mon, 29 Sep 2003 12:41:33 -0700 (PDT) (envelope-from m_evmenkin@yahoo.com) Message-ID: <20030929194133.21012.qmail@web40302.mail.yahoo.com> Received: from [66.35.239.94] by web40302.mail.yahoo.com via HTTP; Mon, 29 Sep 2003 12:41:33 PDT Date: Mon, 29 Sep 2003 12:41:33 -0700 (PDT) From: Maksim Yevmenkin To: John Hay In-Reply-To: <20030929183921.GA88439@zibbi.icomtek.csir.co.za> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii cc: current@freebsd.org Subject: Re: Bluetooth patch X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 29 Sep 2003 19:41:34 -0000 Hi John, > > I have prepared Bluetooth mega patch for FreeBSD source tree. This patch > > updates FreeBSD sources to the most recent snapshot. [...] > > The patch could be downloaded from > > > > http://www.geocities.com/m_evmenkin/patch/bluetooth20030914.diff.gz > > I had a look at the patch and here is my comments. I haven't tried it > yet, but I did try your latest snapshot. good, did it (snapshot) work for you? > I haven't looked at the man > page markup, someone more knowledgable can do that, or it can be > committed and then Ruslan can have a look at it when he gets time. i tried to follow original man page style. it is possible that i missed few minor things, but in general i think it should be fine. my plan was to double check everything after commit and deal with the issues. > There are lots of $FreeBSD$ changes. In a lot of the files that is the only > change. i see, is that a problem? i can clean up the patch and remove these entries. (frankly i thought CVS should take care of it). > The additions in lib/Makefile, share/man/man5/Makefile should be sorted > alphabetically. sure, i assume i should sort entries in lib/Makefile for .if ${MACHINE_ARCH} == "i386" section right? > I think libbluetooth and libsdp should be added to share/mk/bsd.libnames.mk > and then the Makefiles should be modified to use ${LIBBLUETOOTH} and > ${LIBSDP} > on the DPADD lines. /usr/lib/libbluetooth.a should not be hardcoded > otherwise > buildworld won't work correctly. ok, i missed that one :) thanks! > The + after DPADD and LDADD should be removed. It should only be used when > a Makefile have more than one DPADD or LDADD line > > There should not be '-L/usr/lib' on the LDADD line. got it. so, i hope those are minor things. can they be resolved right after commit is done? or i must fix them and submit revised patch? > PS. Will Julian commit it when there was a review or are you looking > for a committer too? Julian and Ruslan are busy at the moment. M. Warner Losh has sent e-mail to core@ and asked for commit bit for me. in the mean time i'd like to commit this and resolve all issues in time for 5.2-RELEASE. thanks, max __________________________________ Do you Yahoo!? The New Yahoo! Shopping - with improved product search http://shopping.yahoo.com