On 03/23/2012 10:51 AM, Wanlong Gao wrote: > Use property to set value instead of use set function directly. > > Signed-off-by: Wanlong Gao <gaowanlong@xxxxxxxxxxxxxx> > --- > virt-clone | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/virt-clone b/virt-clone > index 1199e61..c948cd5 100755 > --- a/virt-clone > +++ b/virt-clone > @@ -70,11 +70,11 @@ def get_clone_macaddr(new_mac, design): > new_mac = None > else: > for i in new_mac: > - design.set_clone_mac(i) > + design.clone_mac = i > > def get_clone_uuid(new_uuid, design): > if new_uuid is not None: > - design.set_clone_uuid(new_uuid) > + design.clone_uuid = new_uuid > > def get_clone_diskfile(new_diskfiles, design, conn, preserve=False, > auto_clone=False): > @@ -115,15 +115,15 @@ def _check_disk(conn, clone_path, orig_path, preserve): > path_to_clone=orig_path) > > def get_clone_sparse(sparse, design): > - design.set_clone_sparse(sparse) > + design.clone_sparse = sparse > > def get_preserve(preserve, design): > - design.set_preserve(preserve) > + design.preserve = preserve > > > def get_force_target(target, design): > for i in target or []: > - design.set_force_target(i) > + design.force_target = i > > def parse_args(): > parser = cli.setupParser() Thanks, pushed now. - Cole
