Re: [PATCH 0/2] client: Add Service API MoveBefore/MoveAfter

2015-06-08 Thread Patrik Flykt
On Wed, 2015-06-03 at 14:19 +0530, Saurav Babu wrote:
 EP-584CD5AB95AE4A3A879836122994DB42
 
 Saurav Babu (2):
   client: Add MoveBefore Service API
   client: Add MoveAfter Service API

Even though ConnMan supports any services to be moved before/after
arbitrary services, the order is not remembered. A VPN can be moved, but
the order for the other services are not remembered after a service list
sort.

Cheers,

Patrik

___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman


Re: [PATCH 0/2] client: Add Service API MoveBefore/MoveAfter

2015-06-08 Thread Saurav Babu
EP-584CD5AB95AE4A3A879836122994DB42

 Saurav Babu (2):
   client: Add MoveBefore Service API
   client: Add MoveAfter Service API

 Even though ConnMan supports any services to be moved before/after
 arbitrary services, the order is not remembered. A VPN can be moved, but
 the order for the other services are not remembered after a service list
 sort.
I added these patches because I found connman client didn?t have support for 
these
Service APIs. I think if the order is not remembered then it would require a 
different
patch in Connman. Do you think that support of MoveBefore/MoveAfter is not 
required to be
added in connman client.

___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman


Re: [PATCH 0/2] client: Add Service API MoveBefore/MoveAfter

2015-06-08 Thread Patrik Flykt
On Mon, 2015-06-08 at 18:52 +0530, Saurav Babu wrote:
 I added these patches because I found connman client didn?t have
 support for these Service APIs. I think if the order is not remembered
 then it would require a different patch in Connman. Do you think that
 support of MoveBefore/MoveAfter is not required to be added in connman
 client.

Supporting this in connmanctl can be useful for VPN cases, but
fixing/implementing this in ConnMan is not going to be easy. Connmanctl
could have this feature if seen useful.

Cheers,

Patrik


___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman


[PATCH 0/2] client: Add Service API MoveBefore/MoveAfter

2015-06-03 Thread Saurav Babu
EP-584CD5AB95AE4A3A879836122994DB42

Saurav Babu (2):
  client: Add MoveBefore Service API
  client: Add MoveAfter Service API

 client/commands.c | 123 ++
 1 file changed, 123 insertions(+)

-- 
1.9.1

___
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman