Skip to content

Commit 978713e

Browse files
committed
Start replacing sed commands with python
It turns out that OSX uses BSD sed and our merge script was written for gnu sed. For portability, let's just convert all sed commands to Python.
1 parent fc97767 commit 978713e

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

utils/merge-rulesets.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,11 @@
6464
def rulesize():
6565
return len(open(rulesets_fn).read())
6666

67+
def clean_up(rulefile):
68+
"""Remove extra whitespace and comments from ruleset library"""
69+
comment_and_newline_pattern = re.compile(r"<!--.*?-->|\n|\r", flags=re.DOTALL)
70+
rulefile = comment_and_newline_pattern.sub('', rulefile)
71+
6772
crush = rulesize()
6873
sedcmd = ["sed", "-i", "-e", ":a", "-re"]
6974
call(sedcmd + [r"s/<!--.*?-->//g;/<!--/N;//ba", rulesets_fn])

0 commit comments

Comments
 (0)