<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Apr 6, 2012, at 3:41 AM, Markus Kittenberger wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><br><br><div class="gmail_quote">On Fri, Apr 6, 2012 at 9:03 AM, Ferry Huberts <span dir="ltr"><<a href="mailto:mailings@hupie.com">mailings@hupie.com</a>></span> <br>
<br><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; ">
I agree.<br>
2.4 is _completely_ ancient and not worth the effort to maintain.<br></blockquote><div>btw its not about 2.4!<br>(which olsrd definetley gets maintaned/tested/used on) as there are still hundreds of router i know of running with 2.4 </div>
<div><br></div><div>as netlink is around since 2.2!!!! </div><div><br></div><div>so for dropping ioctl we where talking about much older linux than 2.4 (-;</div><div><br></div><div>Markus</div></div>
</blockquote></div><br><div>While we are at it, I'd like to suggest explicitly listing the files to be built in the Linux Makefile rather than using a wildcard. A build system is there to make sure the build is repeatable and reliable. The presence or absence of any given file needed for olsrd is something that the build system should be aware of. The only way I know to do that is to list out the files that the build system should be aware of. Something like this:</div><div><br></div><div><br></div><div><div>--- a/make/Makefile.linux</div><div>+++ b/make/Makefile.linux</div><div>@@ -6,8 +6,14 @@ ARCH := $(shell uname -m)</div><div> DESTDIR ?=</div><div> LIBDIR = $(DESTDIR)/usr/lib</div><div> </div><div>-SRCS += $(wildcard src/linux/*.c src/unix/*.c)</div><div>-HDRS += $(wildcard src/linux/*.h src/unix/*.h)</div><div>+SRCS += src/linux/apm.c \</div><div>+ src/linux/kernel_routes_nl.c \</div><div>+ src/linux/kernel_tunnel.c \</div><div>+ src/linux/link_layer.c \</div><div>+ src/linux/net.c \</div><div>+ src/unix/ifnet.c \</div><div>+ src/unix/log.c \</div><div>+ src/unix/misc.c</div><div> </div><div> CPPFLAGS += -Dlinux -DLINUX_NETLINK_ROUTING</div><div> LIBS += </div></div><div><br></div><div><br></div><div>.hc</div><div><br></div><div><br></div></body></html>