On 2012年04月18日 15:44, Guannan Ren wrote:
On 04/18/2012 01:49 PM, Osier Yang wrote:
> As the main excutable program, it's habit to have no suffix.
>
> s/libvirt-test-api.py/libvirt-test-api/g
> ---
> libvirt-test-api.py => libvirt-test-api | 18 +++++++++---------
> 1 files changed, 9 insertions(+), 9 deletions(-)
> rename libvirt-test-api.py => libvirt-test-api (95%)
>
> diff --git a/libvirt-test-api.py b/libvirt-test-api
> similarity index 95%
> rename from libvirt-test-api.py
> rename to libvirt-test-api
> index ae53d14..409f260 100644
> --- a/libvirt-test-api.py
> +++ b/libvirt-test-api
> @@ -33,7 +33,7 @@ from activityfilter import Filter
> from casecfgcheck import CaseCfgCheck
>
> def usage():
> - print "Usage: libvirt-test-api.py<OPTIONS> <ARGUMENTS>"
> + print "Usage: libvirt-test-api<OPTIONS> <ARGUMENTS>"
> print "\noptions: -h, --help : Display usage information \
> \n -c, --casefile: Specify configuration file \
> \n -t, --template: Print testcase config file template \
> @@ -45,14 +45,14 @@ def usage():
>
>
> print "example: \
> - \n python libvirt-test-api.py -l 0|1 -c TEST.CONF \
> - \n python libvirt-test-api.py -c TEST.CONF -f TEST.XML \
> - \n python libvirt-test-api.py -t repos/domain/start.py ... \
> - \n python libvirt-test-api.py -m TESTONE.XML TESTTWO.XML \
> - \n python libvirt-test-api.py -d TEST.XML TESTRUNID TESTID \
> - \n python libvirt-test-api.py -d TEST.XML TESTRUNID \
> - \n python libvirt-test-api.py -d TEST.XML all \
> - \n python libvirt-test-api.py -f TEST.XML \
> + \n python libvirt-test-api -l 0|1 -c TEST.CONF \
> + \n python libvirt-test-api -c TEST.CONF -f TEST.XML \
> + \n python libvirt-test-api -t repos/domain/start.py ... \
> + \n python libvirt-test-api -m TESTONE.XML TESTTWO.XML \
> + \n python libvirt-test-api -d TEST.XML TESTRUNID TESTID \
> + \n python libvirt-test-api -d TEST.XML TESTRUNID \
> + \n python libvirt-test-api -d TEST.XML all \
> + \n python libvirt-test-api -f TEST.XML \
> -r TESTRUNID TESTID ..."
>
> def append_path():
ACK, isn't it better to give the executable permission.
Yeah, I think it's fine to "chmod +x" for it. It will be
in a separate patch.
Guannan Ren
--
libvir-list mailing list
libvir-list(a)redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list