vmbuilder team mailing list archive

[Loïc Minier <lool@xxxxxxxx>]

Loïc Minier has proposed merging lp:~lool/vmbuilder/fix-run-script-group-perms into lp:vmbuilder.

Requested reviews:
  VMBuilder (vmbuilder)


Fix permissions of run.sh script.
-- 
https://code.launchpad.net/~lool/vmbuilder/fix-run-script-group-perms/+merge/36541
Your team VMBuilder is requested to review the proposed merge of lp:~lool/vmbuilder/fix-run-script-group-perms into lp:vmbuilder.

=== modified file 'VMBuilder/plugins/kvm/vm.py'
--- VMBuilder/plugins/kvm/vm.py	2010-06-15 19:55:36 +0000
+++ VMBuilder/plugins/kvm/vm.py	2010-09-24 12:01:05 +0000
@@ -54,7 +54,7 @@
         fp = open(script, 'w')
         fp.write("#!/bin/sh\n\nexec %s\n" % ' '.join(self.cmdline))
         fp.close()
-        os.chmod(script, stat.S_IRWXU | stat.S_IRWXU | stat.S_IROTH | stat.S_IXOTH)
+        os.chmod(script, stat.S_IRWXU | stat.S_IRWXG | stat.S_IROTH | stat.S_IXOTH)
         self.call_hooks('fix_ownership', script)
 
     def libvirt_domain_type_name(self):