From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH v2 0/3] Patch Description
Date: Thu, 14 Oct 2010 06:23:49 -0500 [thread overview]
Message-ID: <4CB6E845.7030605@gmail.com> (raw)
In-Reply-To: <CE761E84DADF2947A4AF22FB8D97A473301905B3@shsmsx501.ccr.corp.intel.com>
[-- Attachment #1: Type: text/plain, Size: 1007 bytes --]
Hi Yang,
On 10/14/2010 06:17 AM, Gu, Yang wrote:
> Hi Denis,
>
>
>>> Then we set the path of script and run the script with its name
>>> dbus-send --system --print-reply --dest=org.phonesim /
>> org.phonesim.Script.SetPath string:/tmp/call
>>> dbus-send --system --print-reply --dest=org.phonesim / org.phonesim.Script.Run
>> string:call.js
>>>
>>
>> One of the changes I made was to have phonesim register to the system
>> bus. This makes the above DBus configuration file unnecessary. Running
>> phonesim on the system bus was too much of a security risk, and wasn't
>> really needed.
>>
>
> Thanks for the modifications afterward, and I'm fine with most of them. One question is why you renamed the service name to "org.ofono.phonesim". I wonder why the phonesim should have relationship with ofono?
In theory the service name should be related to the project website /
domain name. Since we do not own org.phonesim, we put it on the
org.ofono prefix.
Regards,
-Denis
prev parent reply other threads:[~2010-10-14 11:23 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-30 17:52 [PATCH v2 0/3] Patch Description Yang Gu
2010-09-30 17:52 ` [PATCH v2 1/3] Move def of class ControlWidget to header file Yang Gu
2010-09-30 17:52 ` [PATCH v2 2/3] Some instances in ControlWidget need to be deleted Yang Gu
2010-09-30 17:52 ` [PATCH v2 3/3] Enable Qt Script Yang Gu
2010-10-13 20:19 ` [PATCH v2 0/3] Patch Description Denis Kenzior
2010-10-14 11:17 ` Gu, Yang
2010-10-14 11:23 ` Denis Kenzior [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4CB6E845.7030605@gmail.com \
--to=denkenz@gmail.com \
--cc=ofono@ofono.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.