[PATCH 1/2] simple-agent: Fix bug to support multiple parameters in Answer

Jukka Rissanen jukka.rissanen at linux.intel.com
Tue Jan 10 02:55:20 PST 2012


Hi Jeff,

On 01/10/2012 09:09 AM, Jeff Zheng wrote:
> Remove the break statement in order to deal with next argument
> ---
> test/simple-agent | 13 ++++---------
> 1 files changed, 4 insertions(+), 9 deletions(-)
>
> diff --git a/test/simple-agent b/test/simple-agent
> index cda9693..15c2480 100755
> --- a/test/simple-agent
> +++ b/test/simple-agent
> @@ -30,18 +30,15 @@ class Agent(dbus.service.Object):
> args = raw_input('Answer: ')
>
> for arg in args.split():
> - if arg.startswith("Identity="):
> + elif arg.startswith("Identity="):

This looks like a syntax error that you are fixing in patch #2.
You should not do it like this, please make sure that simple-agent
also works after applying patch #1.


Cheers,
Jukka


More information about the connman mailing list