Hi Mohamed,

> diff --git a/test/connect-provider b/test/connect-provider
> new file mode 100755
> index 0000000..8771ff8
> --- /dev/null
> +++ b/test/connect-provider
> @@ -0,0 +1,29 @@
> +#!/usr/bin/python
> +
> +import sys
> +import dbus
> +
> +if (len(sys.argv) < 4):
> +     print "Usage: %s <name> <host> <cookie> [cafile]" % (sys.argv[0])
> +     sys.exit(1)

I think if we call this connect-vpn it is better for now.

> +
> +print "Service path is %s" %(path)

And it is Provider path ;)

> diff --git a/test/list-provider b/test/list-provider
> new file mode 100755
> index 0000000..d808fa0
> --- /dev/null
> +++ b/test/list-provider
> @@ -0,0 +1,27 @@
> +#!/usr/bin/python

It should be named list-providers (plural).

> diff --git a/test/remove-provider b/test/remove-provider
> new file mode 100755
> index 0000000..2367f13
> --- /dev/null
> +++ b/test/remove-provider
> @@ -0,0 +1,22 @@
> +#!/usr/bin/python
> +

Leave this one out for now. And don't forget to update Makefile.am to
include them via EXTRA_DIST.

Regards

Marcel


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

Reply via email to