[PATCH] D57338: gn build: Make cmake sync script work on Windows if git is a bat file
Hans Wennborg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 28 11:34:52 PST 2019
hans accepted this revision.
hans added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/utils/gn/build/sync_source_lists_from_cmake.py:22
+ gn_files = subprocess.check_output(['git', 'ls-files', '*BUILD.gn'],
+ shell=os.name == 'nt').splitlines()
----------------
Maybe worth a comment that shell=True is for the .bat file case?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57338/new/
https://reviews.llvm.org/D57338
More information about the llvm-commits
mailing list