Abpostelnicu added a comment. In https://reviews.llvm.org/D49851#1203676, @JonasToth wrote:
> In https://reviews.llvm.org/D49851#1202764, @Abpostelnicu wrote: > > > Strangely I haven't had those kind of issues but since you propose those > > modifications I would do one more modification, let's output everything to > > stdout and not stderr by doing: > > > > if err: > > sys.stdout.write(str(err) + '\n') > > > > > You can make this a new revision, fixing the `byte` and `str` issues would be > more important now. > > The `byte` and `str` thingie is since the whole python3 releases or did it > change? I'm not sure this is the fix for this, I think we should specify the encoding type when building the string from the byte like: if err: sys.stdout.write(str(err, 'utf-8') + '\n') And of course this must be done only on python 3+ since on python 2 the str ctor doesn't accept the encoding parameter. Repository: rL LLVM https://reviews.llvm.org/D49851 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
