Hi Tomasz,

> Makefile.am    |    3 ++-
>  test/set-proxy |   41 +++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 43 insertions(+), 1 deletions(-)
>  create mode 100755 test/set-proxy
> 
> diff --git a/Makefile.am b/Makefile.am
> index 4e6ecf1..a00464a 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -184,7 +184,8 @@ test_scripts = test/get-state test/list-profiles 
> test/list-services \
>               test/monitor-manager test/test-counter test/set-ip-method \
>               test/set-nameservers test/set-domains test/find-service \
>               test/get-services test/get-proxy-autoconfig \
> -             test/enable-tethering test/disable-tethering test/backtrace
> +             test/enable-tethering test/disable-tethering test/backtrace \
> +             test/set-proxy

the name is so short, just add it to the line with
test/get-proxy-autoconfig ;) 

> if TEST
>  testdir = $(pkglibdir)/test
> diff --git a/test/set-proxy b/test/set-proxy
> new file mode 100755
> index 0000000..5884d27
> --- /dev/null
> +++ b/test/set-proxy
> @@ -0,0 +1,41 @@
> +#!/usr/bin/python
> +
> +import sys
> +import dbus
> +
> +if (len(sys.argv) < 2):
> +     print "Usage: %s <service> [direct|auto|manual] [url=pac-url] 
> [servers=uri1,uri2,...] [excludes=host1,host2,...]" % (sys.argv[0])
> +     print "Example: %s service0 manual servers=proxy.example.com:8080" % 
> sys.argv[0]
> +     print "         This would set the proxy uri and the method to manual"
> +     sys.exit(1)

I prefer if we do the usage like this:

        set-proxy <service> direct
        set-proxy <service> manual <server,[server2,...] 
[exclude,[exclude2,...]]
        set-proxy <service> auto [pac-url]

Not that it matters much, but it is a bit more intuitive.

Regards

Marcel


_______________________________________________
connman mailing list
connman@connman.net
http://lists.connman.net/listinfo/connman

Reply via email to