<div dir="auto">I used the configure switch since SADR makes changes to the LSAs and it will slightly impact network traffic even with source 0::/0. If someone doesn't want that, they can disable it.<div dir="auto"><br></div><div dir="auto">About the name, you are right, it should be changed.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 7 Mar 2017 10:32, "Toke Høiland-Jørgensen" <<a href="mailto:toke@toke.dk">toke@toke.dk</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text">Dean Luga <<a href="mailto:dluga93@gmail.com">dluga93@gmail.com</a>> writes:<br>
<br>
> From: dean <<a href="mailto:dluga93@gmail.com">dluga93@gmail.com</a>><br>
><br>
> It compiles with the macro SADR_OSPF defined.<br>
<br>
</div>Quick comment (I'll look over the rest in more detail later): Why is<br>
source address routing guarded by a configure switch? Shouldn't Bird be<br>
able to understand source-specific routing always, and have the use of<br>
them be configurable at runtime?<br>
<div class="quoted-text"><br>
><br>
> +if test "$enable_sadr" = yes ; then<br>
> +  CPPFLAGS="$CPPFLAGS -D SADR_OSPF"<br>
> +fi<br>
> +<br>
<br>
</div>See above; but if you *are* going to have a config flag, should it maybe<br>
be something that doesn't have the name of a protocol in it (since there<br>
will be other protocols supporting SADR routing)? :)<br>
<font color="#888888"><br>
-Toke<br>
</font></blockquote></div><br></div>