python3 Tools/Scripts/check-webkit-style in dir /Volumes/Data/worker/Style-EWS/build (timeout 1200 secs) watching logfiles {} argv: [b'python3', b'Tools/Scripts/check-webkit-style'] using PTY: False Traceback (most recent call last): File "Tools/Scripts/check-webkit-style", line 46, in <module> sys.exit(CheckWebKitStyle().main()) File "/Volumes/Data/worker/Style-EWS/build/Tools/Scripts/webkitpy/style/main.py", line 159, in main patch_checker.check(patch) File "/Volumes/Data/worker/Style-EWS/build/Tools/Scripts/webkitpy/style/patchreader.py", line 86, in check self._text_file_reader.process_file(file_path=path, line_numbers=line_numbers) File "/Volumes/Data/worker/Style-EWS/build/Tools/Scripts/webkitpy/style/filereader.py", line 137, in process_file self._processor.process(lines, file_path, **kwargs) File "/Volumes/Data/worker/Style-EWS/build/Tools/Scripts/webkitpy/style/checker.py", line 1038, in process checker.check(lines) File "/Volumes/Data/worker/Style-EWS/build/Tools/Scripts/webkitpy/style/checkers/jstest.py", line 104, in check self.check_js_test_functions() File "/Volumes/Data/worker/Style-EWS/build/Tools/Scripts/webkitpy/style/checkers/jstest.py", line 125, in check_js_test_functions baseline_function_map = map_functions_to_dict(baseline_content) File "/Volumes/Data/worker/Style-EWS/build/Tools/Scripts/webkitpy/style/checkers/jstest.py", line 63, in map_functions_to_dict functions = re.split(r'^function\s+', content, flags=re.MULTILINE) File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.7/lib/python3.7/re.py", line 213, in split return _compile(pattern, flags).split(string, maxsplit) TypeError: cannot use a string pattern on a bytes-like object Seen on https://ews-build.webkit.org/#/builders/6/builds/49707, which was testing the patch for https://bugs.webkit.org/show_bug.cgi?id=226902
rdar://79166108
Probably need to change some of these regexes to byte regex
Did this get fixed via bug 226925?
(In reply to Alexey Proskuryakov from comment #3) > Did this get fixed via bug 226925? Yep, I guess Jonathan used that bug instead. *** This bug has been marked as a duplicate of bug 226925 ***