aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimon Warta <[email protected]>2017-07-30 16:59:06 +0200
committerSimon Warta <[email protected]>2017-07-30 17:16:56 +0200
commit7d2d2732d47f2ed9d1b12d265df311cc9904b59f (patch)
treebf5a90cde69ceb3c3e0fa74c08efb5f64b3619e4
parent8f471a00051522c4ca92387aa84dd61e613cf20c (diff)
install.py: pull out calculate_exec_mode
-rwxr-xr-xsrc/scripts/install.py17
1 files changed, 10 insertions, 7 deletions
diff --git a/src/scripts/install.py b/src/scripts/install.py
index f58b171e9..45678d0a7 100755
--- a/src/scripts/install.py
+++ b/src/scripts/install.py
@@ -80,6 +80,15 @@ def force_symlink(target, linkname):
raise e
os.symlink(target, linkname)
+def calculate_exec_mode(options):
+ out = 0o777
+ if 'umask' in os.__dict__:
+ umask = int(options.umask, 8)
+ logging.debug('Setting umask to %s' % oct(umask))
+ os.umask(int(options.umask, 8))
+ out &= (umask ^ 0o777)
+ return out
+
def main(args=None):
if args is None:
args = sys.argv
@@ -89,13 +98,7 @@ def main(args=None):
(options, args) = parse_command_line(args)
- exe_mode = 0o777
-
- if 'umask' in os.__dict__:
- umask = int(options.umask, 8)
- logging.debug('Setting umask to %s' % oct(umask))
- os.umask(int(options.umask, 8))
- exe_mode &= (umask ^ 0o777)
+ exe_mode = calculate_exec_mode(options)
def copy_file(src, dst):
logging.debug('Copying %s to %s' % (src, dst))