| commit | b745acf35cc587034bb9db47cf11e9ac2c021df2 | [log] [tgz] |
|---|---|---|
| author | Jeremy Leconte <jleconte@webrtc.org> | Wed Mar 19 10:44:33 2025 |
| committer | WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> | Wed Mar 19 11:12:40 2025 |
| tree | 2e26e19bc2aeb4b843dbe2d8dd32cc2675d6a3a4 | |
| parent | 03b68808694fdc3282f89805dbb36f8bdc7031ff [diff] |
Revert "Work around an issue with clang-include-cleaner." This reverts commit 89a552a5a3bf508d02c50094528092fb387dc140. Reason for revert: issue as been fixed (https://github.com/llvm/llvm-project/issues/110843) Original change's description: > Work around an issue with clang-include-cleaner. > > Problem has been reported here: > https://github.com/llvm/llvm-project/issues/110843 > > Change-Id: Iaa578a17a724a80ea350db1494229c5af4c454b3 > Bug: b/236227627 > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/364463 > Reviewed-by: Harald Alvestrand <hta@webrtc.org> > Commit-Queue: Jeremy Leconte <jleconte@webrtc.org> > Cr-Commit-Position: refs/heads/main@{#43154} Bug: b/236227627 Change-Id: I5e48300219e62bcd801dd4a49c94554828cce15e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/381661 Reviewed-by: Harald Alvestrand <hta@webrtc.org> Commit-Queue: Jeremy Leconte <jleconte@webrtc.org> Cr-Commit-Position: refs/heads/main@{#44160}
diff --git a/tools_webrtc/iwyu/apply-include-cleaner b/tools_webrtc/iwyu/apply-include-cleaner index 6967bf4..a809e3b 100755 --- a/tools_webrtc/iwyu/apply-include-cleaner +++ b/tools_webrtc/iwyu/apply-include-cleaner
@@ -107,11 +107,7 @@ HAS_OUTPUT=false for FILE in "$@" do - # Run the command in $WORKDIR because of this reported issue: - # https://github.com/llvm/llvm-project/issues/110843 - cd $WORKDIR - OUTPUT=$(../../$CLEANER $INCLUDE_ARGS $COMMAND ../../$FILE) - cd - + OUTPUT=$($CLEANER -p $WORKDIR $INCLUDE_ARGS $COMMAND $FILE) # include-cleaner does not support custom mappings for certain deps # this ensures that the gtest/gmock deps it inserts are replaced