Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-21 Thread Marcel Holtmann
Hi Antti, > > > > please don't implement org.ofono interface in timed. Just implement a > > > > timed specific (com.nokia.time) interface and punch the whole for that > > > > in the timed D-Bus policy. > > > > > > I think there is some disconnect here. The intention is to implement a > > > simila

Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-21 Thread Antti Paila
Hi, On Thu, 2011-01-20 at 12:19 +0100, ext Marcel Holtmann wrote: > Hi Aki, > > > > please don't implement org.ofono interface in timed. Just implement a > > > timed specific (com.nokia.time) interface and punch the whole for that > > > in the timed D-Bus policy. > > > > I think there is some di

Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-20 Thread Marcel Holtmann
Hi Aki, > > please don't implement org.ofono interface in timed. Just implement a > > timed specific (com.nokia.time) interface and punch the whole for that > > in the timed D-Bus policy. > > I think there is some disconnect here. The intention is to implement a > similar agent interface as PushN

Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-19 Thread Aki Niemi
Hi Marcel, 2011/1/18 Marcel Holtmann : > please don't implement org.ofono interface in timed. Just implement a > timed specific (com.nokia.time) interface and punch the whole for that > in the timed D-Bus policy. I think there is some disconnect here. The intention is to implement a similar agent

Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-18 Thread Marcel Holtmann
Hi Aki, > >> So these are redundant, unless the whole thing is rewritten as: > >> > >>>> send_interface="org.ofono.SimToolkitAgent"/> > >>>> send_interface="org.ofono.PushNotificationAgent"/> > >>>> send_interface="org.ofono.SmartMessagingAgent"/> > >> > > > > The agents ar

Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-17 Thread Aki Niemi
Hi, 2011/1/17 Denis Kenzior : >> So these are redundant, unless the whole thing is rewritten as: >> >>       > send_interface="org.ofono.SimToolkitAgent"/> >>       > send_interface="org.ofono.PushNotificationAgent"/> >>       > send_interface="org.ofono.SmartMessagingAgent"/> >> > > The agents ar

Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-17 Thread Denis Kenzior
Hi Mika, > So these are redundant, unless the whole thing is rewritten as: > >send_interface="org.ofono.SimToolkitAgent"/> >send_interface="org.ofono.PushNotificationAgent"/> >send_interface="org.ofono.SmartMessagingAgent"/> > The agents are registered by the application

RE: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-17 Thread Mika.Liljeberg
Hi, On a side topic, the policy seems a bit broken to begin with. > > > > > > > > > > > > > > > > > > This alone already allows root access to all interfaces behind a connection named org.ofono. > > > > > > > > > > > > So these are redundant, unless the wh

Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-17 Thread Marcel Holtmann
Hi Antti, > > > Makefile.am|9 +++-- > > > src/ofono.conf |2 ++ > > > 2 files changed, 9 insertions(+), 2 deletions(-) > > > > > > > > > > > > > > > + > > > > > > > > > > > > > > > + > > > > > > > > > > > > > I am now lost

Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-17 Thread Antti Paila
Hi Marcel, On Fri, 2011-01-14 at 01:22 +0100, ext Marcel Holtmann wrote: > Hi Antti, > > > Makefile.am|9 +++-- > > src/ofono.conf |2 ++ > > 2 files changed, 9 insertions(+), 2 deletions(-) > > > > > > > > > + > > > > > > > > > > + >

Re: [PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-13 Thread Marcel Holtmann
Hi Antti, > Makefile.am|9 +++-- > src/ofono.conf |2 ++ > 2 files changed, 9 insertions(+), 2 deletions(-) > > > + > > > > > + > > > I am now lost in what you are trying to achieve here. I thought we agreed that we send a D

[PATCH 3/5 v3] nettime: DBUS and compilation configuration

2011-01-13 Thread Antti Paila
--- Makefile.am|9 +++-- src/ofono.conf |2 ++ 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/Makefile.am b/Makefile.am index a68fc36..4e30cc9 100644 --- a/Makefile.am +++ b/Makefile.am @@ -321,6 +321,9 @@ builtin_modules += example_nettime builtin_sources += examp