<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>Shouldn't my olsrd.conf plugin parameter now be the location of the commotion socket (if the default is not okay) instead of specifying the serval keyring path? I thought that commotiond contained the necessary bits to hide all the serval specific "stuff" from the end user? </p>

<p>Oh, but now I see below where we need to grab the key from the serval keyring to get things started. </p>

<p>In either case, it seems like it might be nice to have an option to allow the user to customize the path to the commotion socket. </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#r8461003'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/3074564__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwMjk2MzQ4OSwiZGF0YSI6eyJpZCI6MjIzMTA1MjF9fQ==--5b864bf6b2ec0ae541e5a88520c8be7de9e428fb.gif' height='1' width='1'></p>