forked from OSchip/llvm-project
[clang-format] ensure dump_format_style.py works with Python3 correctly
Summary: Python2 has been removed from cygwin, this means anyone running the dump_format_style.py in a cygwin shell could pick up python3 instead In Python3 all strings are unicode as the file is opened in binary mode we need to encode the contents string or we'll face the following error ``` Traceback (most recent call last): File "./dump_format_style.py", line 228, in <module> output.write(contents) TypeError: a bytes-like object is required, not 'str' ``` Reviewed By: krasimir Subscribers: cfe-commits Tags: #clang, #clang-format Differential Revision: https://reviews.llvm.org/D79326
This commit is contained in:
parent
305a4abfd3
commit
3125aa9959
|
@ -225,4 +225,4 @@ contents = open(DOC_FILE).read()
|
|||
contents = substitute(contents, 'FORMAT_STYLE_OPTIONS', options_text)
|
||||
|
||||
with open(DOC_FILE, 'wb') as output:
|
||||
output.write(contents)
|
||||
output.write(contents.encode())
|
||||
|
|
Loading…
Reference in New Issue