Merge pull request #1064 from LLNL/bugfix/github-1043-script-backup

do not create shebang.bak file
This commit is contained in:
Todd Gamblin 2016-06-20 10:45:42 -07:00 committed by GitHub
commit 42ff1aaa75

View file

@ -24,7 +24,6 @@
##############################################################################
import os
from llnl.util.filesystem import *
import llnl.util.tty as tty
import spack
@ -34,6 +33,7 @@
# here, as it is the shortest I could find on a modern OS.
shebang_limit = 127
def shebang_too_long(path):
"""Detects whether a file has a shebang line that is too long."""
with open(path, 'r') as script:
@ -57,16 +57,10 @@ def filter_shebang(path):
if original.startswith(new_sbang_line):
return
backup = path + ".shebang.bak"
os.rename(path, backup)
with open(path, 'w') as new_file:
new_file.write(new_sbang_line)
new_file.write(original)
copy_mode(backup, path)
unset_executable_mode(backup)
tty.warn("Patched overly long shebang in %s" % path)