<div>thx for the reminder,..<br></div><div><div>but my patch never was a build break, as it already did not compile without -D LINUX_NETLINK_ROUTING </div><br></div><div>but i/we should really either remove kernel_routes_ioctl.c or fix it. (could push a fix, but will NOT do so,..)</div>
<div>but i still think removing it makes more sense.</div><div><br></div><div>as imho, the non netlink code likely did not work since a long time (and like the bsd code, it should have issues due to the wrong flags set in olsr_rt_flags())</div>
<div><br></div><div>furthermore even with a fixed kernel_routes_ioctl.c olsrd did not compile without -D LINUX_NETLINK_ROUTING</div><div><br></div><div>as e.g. the smartgateway code in not only within #ifdef LINUX_NETLINK_ROUTING </div>
<div>therefore since olsrd 0.6.0 olsrd did not compile without LINUX_NETLINK_ROUTING</div><div><br></div><div>while this was easy fixable too, and i could compile and run it, txtinfo did not work (or it crashes olsrd once he tries to load it) too.</div>
<div>furthermore it would now make sense to test the ioctl routing code, for several issues we only fixed for netlink,..</div><div><br>so i finally decided (with henning) NOT to fix it, as he will remove linux_ioctl stuff prior to olsrd 0.6.3 release,.. ( thx Henning! )</div>
<div><br></div><div>Markus</div><div><br></div><div class="gmail_quote">On Thu, Apr 5, 2012 at 7:56 PM, Hans-Christoph Steiner <span dir="ltr"><<a href="mailto:hans@guardianproject.info">hans@guardianproject.info</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I just noticed something while rebasing my patches against the HEAD of stable.  In the commit called "fix bsd RTM_GATEWAY routing flags detection" by Markus Kittenberger (66dcc5f8e3da8084cef0fcec3a4580d47b90f593), the function olsr_rt_flags() was changed to have a second parameter:<br>

<br>
  olsr_rt_flags(const struct rt_entry *, int add);<br>
<br>
This change was made in src/process_routes.h,  src/process_routes.c, and src/bsd/kernel_routes.c but was not made in src/linux/kernel_routes_ioctl.c.  Yet, olsrd seems to build and run fine.  My guess is that src/linux/kernel_routes_ioctl.c is not used at all, even though its being included it the build.<br>
</blockquote><div>it is only used if one want olsrd without netlink for linux,..</div><div><br></div><div>and yes either i should change kernel_routes_ioctl.c (as i already promised)</div><div>or remove it,..</div><div><br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
If src/linux/kernel_routes_ioctl.c is not used, can it be removed?  If it is, then olsr_rt_flags() should be updated there.<br>
<br>
On a related note, I setup an automated build on Debian/stable/amd64 and also runs olsrd using fakeroot after it builds:<br>
<a href="https://build.safermobile.org/job/olsrd-stable/" target="_blank">https://build.safermobile.org/job/olsrd-stable/</a><br>
<span class="HOEnZb"><font color="#888888"><br>
.hc</font></span><br>--<br>
Olsr-dev mailing list<br>
<a href="mailto:Olsr-dev@lists.olsr.org">Olsr-dev@lists.olsr.org</a><br>
<a href="https://lists.olsr.org/mailman/listinfo/olsr-dev" target="_blank">https://lists.olsr.org/mailman/listinfo/olsr-dev</a><br></blockquote></div><br>