aaron.ballman added a comment.

This seems reasonable to me except for the fact that the script doesn't run 
when I try it locally.

  c:\llvm\tools\clang\tools\extra\docs\clang-tidy\checks>python 
gen-static-analyzer-docs.py
  Traceback (most recent call last):
    File "gen-static-analyzer-docs.py", line 148, in <module>
      main()
    File "gen-static-analyzer-docs.py", line 137, in main
      checkers = get_checkers(file_path)
    File "gen-static-analyzer-docs.py", line 22, in get_checkers
      result = subprocess.run(["llvm-tblgen", "--dump-json", "-I",
  AttributeError: 'module' object has no attribute 'run'
  
  c:\llvm\tools\clang\tools\extra\docs\clang-tidy\checks>python --version
  Python 2.7.16

I don't think that API is available in Python 2.7, which is our current min 
required version of Python. That's the last outstanding item for the review 
that I can see.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64454/new/

https://reviews.llvm.org/D64454



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to