**** BEGIN LOGGING AT Sat Sep 15 02:59:57 2007 Sep 15 08:54:42 nbd * r8780 /trunk/Makefile: add 'make prepare' target for building the tools and toolchain Sep 15 13:29:42 Kaloz, ping Sep 15 13:30:19 please move http://forum.openwrt.org/viewtopic.php?id=12541 to 'Community Releases / Announcements'. thanks. Sep 15 16:20:48 nico * r8781 /trunk/include/netfilter.mk: prevent include/netfilter.mk from being included multiple times Sep 15 16:36:00 {Nico}: http://homepage.mac.com/danielg4/rdc.patch -dependency corrections Sep 15 16:40:02 <{Nico}> sn9: i'll keep the '+' for kmod-rt* Sep 15 16:40:48 that doesn't work right Sep 15 16:41:02 without + it works right Sep 15 16:41:33 <{Nico}> because it tends do hide things without it and it prevents other packages from directly selecting a package hidden in a dependency chain Sep 15 16:41:55 no, the opposite Sep 15 16:42:10 + pevents other packages from directly selecting a package hidden in a dependency chain Sep 15 16:42:11 <{Nico}> it will work if i add some @PCI|USB_SUPPORT Sep 15 16:42:38 because the dependency is nullified that way Sep 15 16:42:56 <{Nico}> nope, if i select kmod-rt73usb, it will select everything required Sep 15 16:43:04 +kmod-rt* is the same as no dependency Sep 15 16:44:03 you can select kmod-rt73usb without kmod-rt2x00-usb, which shouldn't happen Sep 15 16:44:11 <{Nico}> this is not really a dependency, it says "if you want this one, you must have theese other as well" Sep 15 16:44:31 that's what dependency means Sep 15 16:46:15 <{Nico}> yep, but without '+' it says "you can't have this one, you must select this one first" Sep 15 16:46:25 <{Nico}> not reallu the same Sep 15 16:46:47 <{Nico}> it tends to hide things away Sep 15 16:46:47 agreed, but that's a build system inadequacy Sep 15 16:47:16 the hiding, also Sep 15 16:47:33 <{Nico}> we don't need that for packages, we just ant to insure that when someone install a package, other packages required for it to work will be installed as well Sep 15 16:47:57 and that's exactly what doesn't happen with + Sep 15 16:48:47 <{Nico}> that's exactly what happens with + Sep 15 16:49:02 i've tried every permutation Sep 15 16:49:37 <{Nico}> selecting kmot-rt73usb with force kmod-rt2x00-usb and the kmod-rt2x00-lib and that's what we want Sep 15 16:49:50 <{Nico}> s/with/will Sep 15 16:50:27 it was not forcing them when i tried it, repeatedly. without +, the forcing happens Sep 15 16:51:16 the hiding is a relatively minor annoyance Sep 15 16:51:16 <{Nico}> without +, there's no forcing implied : you're forcing it by selecting packages first Sep 15 16:52:15 you meant something else by "forcing" here? Sep 15 16:52:50 <{Nico}> the basic rule for me is don't use + for subpackages (plugins, modules and the like) and use + elsewhere Sep 15 16:53:06 these are subpackages Sep 15 16:53:32 <{Nico}> nope, rt2x00-lib alone doesn't do anything Sep 15 16:54:01 right, and it is selectable by itself in any case Sep 15 16:54:07 sn9: afaik, the "+" infront of a dep is used if the package it needs, is into another Makefile, if the package you want to add as dep in there is in the same Makefile, you'll just write the name of the package without the "+" infront of it Sep 15 16:54:22 crazy_imp: exactly Sep 15 16:54:43 <{Nico}> take the case of usb webcam modules, they depends on video and usb Sep 15 16:55:09 yes, so they use + Sep 15 16:55:20 (but it's maybe different in the kernelmodules stuff of the buildroot, dunno) Sep 15 16:56:48 <{Nico}> what would you want 3 level of uneeded dependency Sep 15 16:56:54 <{Nico}> s/what/why Sep 15 16:57:34 <{Nico}> apart from hiding things until someone selects smth Sep 15 16:58:27 so that one cannot do something like select kmod-rt73usb without kmod-rt2x00-usb, which is currently possible Sep 15 16:58:59 <{Nico}> no, it's not possible : i just checked Sep 15 16:59:44 <{Nico}> there are "---" in front of forced packages Sep 15 17:00:10 yes, and that does not appear Sep 15 17:00:29 {Nico}: that's working on your system? Sep 15 17:02:23 interesting, thought it was not working here yesterday Sep 15 17:02:28 *confused* Sep 15 17:03:52 <{Nico}> crazy_imp: http://pastebin.ca/698688 Sep 15 17:05:39 hmm, it works now; it didn't before. never mind -- i guess the build system was temporarily broken at some point Sep 15 17:06:34 {Nico}: ack Sep 15 17:07:26 <{Nico}> sn9: this is the expected behaviour Sep 15 17:07:59 <{Nico}> rt2x00 just misses some @PCI_SUPPORT and @USB_SUPPORT conditionnals Sep 15 17:08:04 i expected it before, yes. it's not what i saw then, though Sep 15 21:03:07 florian * r8782 /trunk/package/linux-atm/ (5 files in 3 dirs): Add support for IP over ATM **** ENDING LOGGING AT Sun Sep 16 02:59:56 2007