[PATCH] D93565: scan-view: Remove Reporter.py and associated AppleScript files

tstellar added a comment. In D93565#2773761 , @puremourning wrote: > Erm, doh. I thought I was on main, but I was using clang 12.0.0 (tag > llvmorg-12.0.0 from llvm-project). We tend to only use release tagged > versions. OK, I've cherry-picked this

[PATCH] D93565: scan-view: Remove Reporter.py and associated AppleScript files

puremourning added a comment. Erm, doh. I thought I was on main, but I was using clang 12.0.0 (tag llvmorg-10.0.0 from llvm-project). We tend to only use release tagged versions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93565/new/

[PATCH] D93565: scan-view: Remove Reporter.py and associated AppleScript files

tstellar added a comment. Are you still seeing failures in the main branch? I thought I had fixed this with Revision: https://reviews.llvm.org/D96367. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93565/new/ https://reviews.llvm.org/D93565

[PATCH] D93565: scan-view: Remove Reporter.py and associated AppleScript files

puremourning added a comment. In D93565#2770709 , @puremourning wrote: > Unless i'm missing something, this change seems to have broken scan-view, as > it now just says > > "ModuleNotFOundError: No module named 'Reporter'" > > Due to : > > `import

[PATCH] D93565: scan-view: Remove Reporter.py and associated AppleScript files

puremourning added a comment. Unless i'm missing something, this change seems to have broken scan-view, as it now just says "ModuleNotFOundError: No module named 'Reporter'" Due to : `import Reporter` in `ScanView.py` Am i losing my mind? Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D93565: scan-view: Remove Reporter.py and associated AppleScript files

This revision was automatically updated to reflect the committed changes. Closed by commit rGdbb01536f6f4: scan-view: Remove Reporter.py and associated AppleScript files (authored by tstellar). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D93565: scan-view: Remove Reporter.py and associated AppleScript files

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Uhm, yeah, indeed. Let's remove it, thanks! I doubt people even use scan-view at this point (but i'd rather keep it) (scan-build is definitely used). Comment at:

[PATCH] D93565: scan-view: Remove Reporter.py and associated AppleScript files

tstellar created this revision. tstellar added reviewers: ddunbar, krememek. Herald added subscribers: Charusso, mgorny. tstellar requested review of this revision. Herald added a project: clang. I'm not exactly sure what this is, but it appears to be a tool for reporting internal issues at