Dean Luga <dlug...@gmail.com> writes: > From: dean <dlug...@gmail.com> > > It compiles with the macro SADR_OSPF defined.
Quick comment (I'll look over the rest in more detail later): Why is source address routing guarded by a configure switch? Shouldn't Bird be able to understand source-specific routing always, and have the use of them be configurable at runtime? > > +if test "$enable_sadr" = yes ; then > + CPPFLAGS="$CPPFLAGS -D SADR_OSPF" > +fi > + See above; but if you *are* going to have a config flag, should it maybe be something that doesn't have the name of a protocol in it (since there will be other protocols supporting SADR routing)? :) -Toke
