[PATCH] D147742: [python] Enable bindings tests on Windows

2023-04-07 Thread Artur Ryt via Phabricator via cfe-commits
R2RT added a comment. According to CI logs, these tests are still not run on Windows builder. Line below, present in Debian's log, is missing. [9746/10613] cd /var/lib/buildkite-agent/builds/llvm-project/clang/bindings/python && /usr/bin/cmake -E env CLANG_NO_DEFAULT_CONFIG=1 CLANG_LIBRARY_P

[PATCH] D147742: [python] Enable bindings tests on Windows

2023-04-06 Thread Artur Ryt via Phabricator via cfe-commits
R2RT created this revision. R2RT added a project: clang. Herald added a project: All. R2RT requested review of this revision. Herald added a subscriber: cfe-commits. Unlike stated by removed comment, they do simply pass in my local enviroment. In order to find necessary fixes (if any), CI has to

[PATCH] D147414: [python] Expose clang_Location_isInSystemHeader

2023-04-03 Thread Artur Ryt via Phabricator via cfe-commits
R2RT added a comment. In D147414#4240343 , @aaron.ballman wrote: > LGTM! Do you need someone to land this on your behalf? If so, please let me > know what name and email address you would like for patch attribution. > > In D147414#4239453

[PATCH] D147414: [python] Expose clang_Location_isInSystemHeader

2023-04-03 Thread Artur Ryt via Phabricator via cfe-commits
R2RT updated this revision to Diff 510542. R2RT added a comment. Bring back Python section in release notes and document new `is_in_system_header` property. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147414/new/ https://reviews.llvm.org/D147414 Files: clang/bindings/python/clang/

[PATCH] D147414: [python] Expose clang_Location_isInSystemHeader

2023-04-02 Thread Artur Ryt via Phabricator via cfe-commits
R2RT added a comment. I thought about adding it into release notes, but recently Python section has got removed from ReleaseNotes.rst, so I am not sure what to do now: https://reviews.llvm.org/D142578 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D147414: [python] Expose clang_Location_isInSystemHeader

2023-04-02 Thread Artur Ryt via Phabricator via cfe-commits
R2RT created this revision. R2RT added a reviewer: aaron.ballman. R2RT added a project: clang. Herald added a subscriber: arphaman. Herald added a project: All. R2RT requested review of this revision. Herald added a subscriber: cfe-commits. Add `is_in_system_header` property for `Location` class.

[PATCH] D49851: [clang-tidy] run-clang-tidy add synchronisation to the output

2018-08-16 Thread Artur Ryt via Phabricator via cfe-commits
R2RT reopened this revision. R2RT added a comment. This revision is now accepted and ready to land. Hey, all, as for me it seems that this commit introduced a few problems with Python3 (at least 3.7) sys.stdout.write(' '.join(invocation) + '\n' + output + '\n') TypeError: can only concatenat