<p>In lib/mdp/src/olsrd_mdp.c:</p>
<pre style='color:#555'>>  
> -  if (!strlen(config_instancepath)) {
> -    olsr_printf(1, "[MDP] Must set a Serval instance path (servalpath) for this plugin to work.\n\n");
> -    exit(1);
> -  }
> +  CHECKF(strlen(config_instancepath),"[MDP] Must set a Serval instance path (servalpath) for this plugin to work.\n\n");
> +  
> +  CHECKF(co_init() == 1,"Failed to initialize Commotion client\n\n");
> +  
> +  CHECKF((co_conn = co_connect(CO_SOCK,sizeof(CO_SOCK))),"Failed to connect to Commotion socket\n\n");
>  
</pre>
<p>yes, adding a config option for the commotion socket would be a great idea. but we still need to get the serval path and SID from config parameters. The nice thing is those values are defined in interface-specific commotion profiles, and then fetched from the commotion daemon with a hotplug script and put into the olsrd-mdp config.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/opentechinstitute/olsrd/pull/15/files#r8469836'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/3074564__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwMjk5NTM1OSwiZGF0YSI6eyJpZCI6MjIzMTA1MjF9fQ==--6a6975e58afbccd46bd84dd5898924e470501113.gif' height='1' width='1'></p>