Message ID | 20170308175407.11142-2-joe@ovn.org |
---|---|
State | Accepted |
Headers | show |
On Wed, Mar 8, 2017 at 12:54 PM, Joe Stringer <joe@ovn.org> wrote: > Signed-off-by: Joe Stringer <joe@ovn.org> > --- > utilities/checkpatch.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py > index e795646de4a8..a94736d380c2 100755 > --- a/utilities/checkpatch.py > +++ b/utilities/checkpatch.py > @@ -185,7 +185,7 @@ def ovs_checkpatch_parse(text): > re.I | re.M | re.S) > skip_line_length_check = False > > - for line in text.split('\n'): > + for line in text.decode().split('\n'): > if current_file != previous_file: > previous_file = current_file > if any([fmt in current_file for fmt in line_length_blacklist]): Acked-by: Russell Bryant <russell@ovn.org>
On 8 March 2017 at 11:25, Russell Bryant <russell@ovn.org> wrote: > On Wed, Mar 8, 2017 at 12:54 PM, Joe Stringer <joe@ovn.org> wrote: >> Signed-off-by: Joe Stringer <joe@ovn.org> >> --- >> utilities/checkpatch.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py >> index e795646de4a8..a94736d380c2 100755 >> --- a/utilities/checkpatch.py >> +++ b/utilities/checkpatch.py >> @@ -185,7 +185,7 @@ def ovs_checkpatch_parse(text): >> re.I | re.M | re.S) >> skip_line_length_check = False >> >> - for line in text.split('\n'): >> + for line in text.decode().split('\n'): >> if current_file != previous_file: >> previous_file = current_file >> if any([fmt in current_file for fmt in line_length_blacklist]): > > Acked-by: Russell Bryant <russell@ovn.org> Thanks, applied to master.
diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py index e795646de4a8..a94736d380c2 100755 --- a/utilities/checkpatch.py +++ b/utilities/checkpatch.py @@ -185,7 +185,7 @@ def ovs_checkpatch_parse(text): re.I | re.M | re.S) skip_line_length_check = False - for line in text.split('\n'): + for line in text.decode().split('\n'): if current_file != previous_file: previous_file = current_file if any([fmt in current_file for fmt in line_length_blacklist]):
Signed-off-by: Joe Stringer <joe@ovn.org> --- utilities/checkpatch.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)