abhina.sreeskantharajan added a comment. In D95246#2566029 <https://reviews.llvm.org/D95246#2566029>, @ASDenysPetrov wrote:
> In D95246#2565351 <https://reviews.llvm.org/D95246#2565351>, > @abhina.sreeskantharajan wrote: > >> > > > >> Do you know what is different between your environments which is causing the >> spelling to be capitalized? This might help me determine how to fix the >> current host platform check in llvm/utils/lit/lit/llvm/config.py. > > I'd love to help you. I'm using ordinary Win10. Here is my python output: > > >>> os.name > 'nt' > >>> sys.platform > 'win32' > >>> platform.system() > 'Windows' > >>> platform.release() > '10' > >>> > > If you would say what exact info you need I would execute and send you. > BTW my python3 uses //Sentence case capitalization// in errors as you can see: > > >>> open('some') > Traceback (most recent call last): > File "<stdin>", line 1, in <module> > FileNotFoundError: [Errno 2] No such file or directory: 'some' > >>> Thanks! The following Windows bot (clang-x64-windows-msvc) is also passing http://lab.llvm.org:8011/#/builders/123, I expect it would have the same output in python as your machine. James mentioned that this issue might be caused by a difference in standard library. I was curious if all of these modified testcases are now failing or only the three you mentioned. I also don't know how this change can cause a pop-up to occur, is there more information you can provide? And lastly, if you have the time, does this testcase pass for you? llvm/test/tools/llvm-elfabi/fail-file-write-windows.test I did not modify this but it requires system-windows and has a lower-case error message. I expect if my change causes failures for you, then this testcase should also fail. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95246/new/ https://reviews.llvm.org/D95246 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
