Hi, sorry for that. I've just double checked the patch and there is no other code outside an #ifdef P2MP_SERVER
Davide. On 08/ott/2011, at 12:31, Gert Doering wrote: > Hi, > > On Fri, Sep 30, 2011 at 11:12:12AM +0200, David Sommerseth wrote: >> On 15/09/11 23:42, Davide Guerri wrote: >>> Please find attached a patch for a new stale-routes-check (and remove) >>> option. >> >> Applied to the master branch in testing and stable trees. > > The way this is currently implemented breaks compilation with > --disable-crypto --disable-ssl (glad we have buildbot to test this :) ), > because in that case, P2MP_SERVER is not enabled, and > options->stale_routes_ageing_time and options->stale_routes_check_interval > are not defined. > > The patch below moves the block inside the #ifdef P2MP_SERVER block > inside options.c - the code is not changed, just moved elsewhere. > > Smoke-tested on FreeBSD 8.2 (just compilation, not even started). > > Please verify, ACK, and commit :-) > > gert > -- > USENET is *not* the non-clickable part of WWW! > //www.muc.de/~gert/ > Gert Doering - Munich, Germany g...@greenie.muc.de > fax: +49-89-35655025 g...@net.informatik.tu-muenchen.de > <0001-move-block-for-stale-routes-check-config-inside-ifde.patch>
