HEADSUP: Bluetooth patch is about to be committed
Dear Hackers, In the next few hours i will be committing Bluetooth patch. The patch was reviewed by M. Warner Losh [EMAIL PROTECTED] and John Hay [EMAIL PROTECTED] After commit i will re-cvsup my system and will do a full buildworld (could take 4-6 hours) to make sure everything works. Expect bumps :) Please feel free to contact me if you have bluetooth/build problems and believe that it is my fault :) thanks, max __ Do you Yahoo!? The New Yahoo! Shopping - with improved product search http://shopping.yahoo.com ___ [EMAIL PROTECTED] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to [EMAIL PROTECTED]
Re: Bluetooth patch
John, [...] : 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. : : I can give it a go if no one else wants to do it. [...] However, if you'd like to help review the changes before/as they go into the tree, my feelings won't be hurt :-) If you need help time wise I'd be happy to help. the revised version of the patch can be downloaded from http://www.geocities.com/m_evmenkin/patch/bluetooth20030930.diff.gz i think, i fixed all the issues you found. please review and let me know if there are any problems. thanks, max __ Do you Yahoo!? The New Yahoo! Shopping - with improved product search http://shopping.yahoo.com ___ [EMAIL PROTECTED] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to [EMAIL PROTECTED]
Re: Bluetooth patch
On Mon, Sep 29, 2003 at 10:51:56PM +0200, John Hay wrote: 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). I don't think it will break cvs, it might just cause some extra bloat. Maybe just get rid of those parts of the patch where the only change to the file is the $FreeBSD$ line? We have CVS magic that unexpands the $FreeBSD$ on it's way back into the repo. It might complain that it is invalid, but it shouldn't cause any repo bloat. pgp0.pgp Description: PGP signature
Re: Bluetooth patch
Hi Maksim, I have prepared Bluetooth mega patch for FreeBSD source tree. This patch updates FreeBSD sources to the most recent snapshot. The patch is quite extensive - it adds two new libraries (libbluetooth and libsdp) as well as puts some files into /etc/bluetooth and modifies quite a few other files. I also have modified Makefile's to add new libraries and usr.{s}bin/bluetooth to the build. I've sent it to Julian and Ruslan, but they do not have free time to look at it. If anyone wants to review the patch please do so and let me know if i missed/forget anything. 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. 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. There are lots of $FreeBSD$ changes. In a lot of the files that is the only change. The additions in lib/Makefile, share/man/man5/Makefile should be sorted alphabetically. 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. 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. PS. Will Julian commit it when there was a review or are you looking for a committer too? John -- John Hay -- [EMAIL PROTECTED] / [EMAIL PROTECTED] ___ [EMAIL PROTECTED] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to [EMAIL PROTECTED]
Re: Bluetooth patch
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 ___ [EMAIL PROTECTED] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to [EMAIL PROTECTED]
Re: Bluetooth patch
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? Yes, I got a bluetooth ppp session going between 2 FreeBSD boxes with usb-bluetooth devices. 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. That is ok. 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). I don't think it will break cvs, it might just cause some extra bloat. Maybe just get rid of those parts of the patch where the only change to the file is the $FreeBSD$ line? 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? Yes, that too, but inside such a block it should be alphabetical. SUBDIR should also be alphabetical except for those mentioned in the comments at the top of src/lib/Makefile. 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? I would prefer a patch with the makefile stuff fixed so that I can push it through a make world and a make release before committing. 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. I can give it a go if no one else wants to do it. John -- John Hay -- [EMAIL PROTECTED] / [EMAIL PROTECTED] ___ [EMAIL PROTECTED] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to [EMAIL PROTECTED]
Re: Bluetooth patch
In message: [EMAIL PROTECTED] John Hay [EMAIL PROTECTED] writes: : 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. : : I can give it a go if no one else wants to do it. The core approval process for committers takes one week, so he should have his commit bit in plenty of time. However, if you'd like to help review the changes before/as they go into the tree, my feelings won't be hurt :-) Warner ___ [EMAIL PROTECTED] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to [EMAIL PROTECTED]
Re: Bluetooth patch
On Mon, Sep 29, 2003 at 05:49:08PM -0600, M. Warner Losh wrote: In message: [EMAIL PROTECTED] John Hay [EMAIL PROTECTED] writes: : 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. : : I can give it a go if no one else wants to do it. The core approval process for committers takes one week, so he should have his commit bit in plenty of time. Hey, I didn't realise the process was that far already. My initial reaction was because it looked like nobody else reacted and I got tired of having to add the bluetooth snap everytime. :-) However, if you'd like to help review the changes before/as they go into the tree, my feelings won't be hurt :-) If you need help time wise I'd be happy to help. John -- John Hay -- [EMAIL PROTECTED] / [EMAIL PROTECTED] ___ [EMAIL PROTECTED] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to [EMAIL PROTECTED]
Bluetooth patch
Dear Hackers, I have prepared Bluetooth mega patch for FreeBSD source tree. This patch updates FreeBSD sources to the most recent snapshot. The patch is quite extensive - it adds two new libraries (libbluetooth and libsdp) as well as puts some files into /etc/bluetooth and modifies quite a few other files. I also have modified Makefile's to add new libraries and usr.{s}bin/bluetooth to the build. I've sent it to Julian and Ruslan, but they do not have free time to look at it. If anyone wants to review the patch please do so and let me know if i missed/forget anything. The patch could be downloaded from http://www.geocities.com/m_evmenkin/patch/bluetooth20030914.diff.gz thanks, max __ Do you Yahoo!? Yahoo! SiteBuilder - Free, easy-to-use web site design software http://sitebuilder.yahoo.com ___ [EMAIL PROTECTED] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to [EMAIL PROTECTED]