Hi,

On Wed, May 18, 2011 at 04:38:21PM +0200, Xavier Franquet wrote:
>    if (!r->defined)
>      return;
> @@ -1113,23 +1114,26 @@ add_route (struct route *r, const struct tuntap *tt, 
> unsigned int flags, const s
>  
>  #if defined(TARGET_LINUX)
>  #ifdef CONFIG_FEATURE_IPROUTE
> -  argv_printf (&argv, "%s route add %s/%d via %s",
> +  argv_printf (&argv, "%s route add %s/%d via %s dev %s",

A few words on why exactly this patch is needed, and what problems it
solves would be appreciated...

As it stands, I'd NAK it, because "the existing code works for me", and 
your code just makes it more complicated.

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

Attachment: pgpiMBI3tjaio.pgp
Description: PGP signature

Reply via email to