Alon Bar-Lev has uploaded a new change for review.
Change subject: packaging: setup: support non standard umask
......................................................................
packaging: setup: support non standard umask
simplest solution for supporting invalid umask is to modify it to known
value.
new setup behaves better in this regard.
Change-Id: If119a60203885fec0debd55e31d2ec8fec03ca79
Signed-off-by: Alon Bar-Lev <alo...@redhat.com>
---
M packaging/fedora/setup/engine-setup.py
M packaging/fedora/setup/engine-upgrade.py
2 files changed, 9 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/96/15496/1
diff --git a/packaging/fedora/setup/engine-setup.py
b/packaging/fedora/setup/engine-setup.py
index 613e4cc..0e4a8e1 100755
--- a/packaging/fedora/setup/engine-setup.py
+++ b/packaging/fedora/setup/engine-setup.py
@@ -2464,6 +2464,11 @@
plugin.initSequences(controller)
def initMain():
+
+ # make sure we create files
+ # as we intend to
+ os.umask(0022)
+
#verify that root is the user executing the script - only supported user
for now.
#TODO: check how we can change this to user rhevm
_verifyUserPermissions()
diff --git a/packaging/fedora/setup/engine-upgrade.py
b/packaging/fedora/setup/engine-upgrade.py
index 609b6d9..88229f0 100755
--- a/packaging/fedora/setup/engine-upgrade.py
+++ b/packaging/fedora/setup/engine-upgrade.py
@@ -1471,6 +1471,10 @@
if __name__ == '__main__':
try:
+ # make sure we create files
+ # as we intend to
+ os.umask(0022)
+
# Change to the root directory to avoid problems if our current
# working directory is deleted:
os.chdir("/")
--
To view, visit http://gerrit.ovirt.org/15496
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If119a60203885fec0debd55e31d2ec8fec03ca79
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
