[PATCH] D111490: [DOCS] Update ninja build doc (new: github link, build command, and chmod requirements)

Shivam Gupta via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 9 10:36:20 PDT 2021


xgupta added inline comments.


================
Comment at: clang/docs/LibASTMatchersTutorial.rst:34
-      git clone https://github.com/martine/ninja.git
-      cd ninja
-      git checkout release
----------------
SamuelMarks wrote:
> xgupta wrote:
> > why removed `cd ninja` & `git checkout release`?
> @xgupta To match how ninja is installed everywhere else in LLVM guides. Also ninja's release branch is almost a year old, whereas its master is 5 days old.
fine, but we need to cd into ninja directory to run configure step, right?


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

https://reviews.llvm.org/D111490



More information about the cfe-commits mailing list