Hi Patrik,

On Wed, Mar 21, 2012 at 2:33 PM, Patrik Flykt
<[email protected]>wrote:

> On Wed, 2012-03-21 at 13:08 +0200, Alok Barsode wrote:
> > diff --git a/plugins/meego.c b/plugins/meego.c
> > index f8d2f0f..903f542 100644
> > --- a/plugins/meego.c
> > +++ b/plugins/meego.c
> > @@ -31,12 +31,12 @@
> >
> >  static int meego_init(void)
> >  {
> > -     return __connman_timeserver_system_append(MEEGO_NTP_SERVER);
> > +     return __connman_timeserver_system_set(MEEGO_NTP_SERVER);
> >  }
> >
> >  static void meego_exit(void)
> >  {
> > -     __connman_timeserver_system_remove(MEEGO_NTP_SERVER);
> > +
> >  }
> >
> >  CONNMAN_PLUGIN_DEFINE(meego, "MeeGo features plugin", VERSION,
>
> We want to get rid of the meego.c plugin as in the TODO file and
> http://lists.connman.net/pipermail/connman/2012-March/008605.html.
> Please make a patch for FallbackTimeservers instead.
>
> This is a bit unclear to me. the meego plugin set the global timeserver to
ntp.meego.com.
Do we want the fallback servers to replace the global timeservers?


> Cheers,
>
>        Patrik
>
> Cheers,
Alok.

> _______________________________________________
> connman mailing list
> [email protected]
> http://lists.connman.net/listinfo/connman
>
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to