[PATCH] D100714: [docs] Add a new tutorial that talk about how to make a change to llvm.

[Shivam Gupta via Phabricator via llvm-commits]

xgupta added a comment.

I think you should upload the patch with full context by `git diff HEAD~1 -U999999 > mypatch.patch`, please see the phabricator documentation <https://llvm.org/docs/Phabricator.html>.

maybe that was the reason of failed to apply patch error. The error I am getting is

Checking patch llvm/docs/GettingStartedTutorials.rst...
error: while searching for:

  GettingStartedVS
  ProgrammersManual
  tutorial/index

:doc:`GettingStarted`

  Discusses how to get up and running quickly with the LLVM infrastructure.

error: patch failed: llvm/docs/GettingStartedTutorials.rst:12
error: while searching for:

  on Windows.

:doc:`CompilerWriterInfo`

  A list of helpful links for compiler writers.

error: patch failed: llvm/docs/GettingStartedTutorials.rst:35
Applied patch llvm/docs/MyFirstTypoFix.rst cleanly.
Applying patch llvm/docs/GettingStartedTutorials.rst with 2 rejects...
Rejected hunk #1.
Rejected hunk #2.

Patch Failed!



================
Comment at: llvm/docs/MyFirstTypoFix.rst:170
+
+   ln -s build/compile_commands.json ../
+
----------------
$ is missing in the beginning.


================
Comment at: llvm/docs/MyFirstTypoFix.rst:229
+
+   vi ../clang/include/clang/Basic/DiagnosticSemaKinds.td
+
----------------
$ is missing in the beginning.


================
Comment at: llvm/docs/MyFirstTypoFix.rst:254
+
+   ninja check-clang
+
----------------
$ is missing in the beginning.


================
Comment at: llvm/docs/MyFirstTypoFix.rst:275
+
+   vi ../clang/test/SemaCXX/warn-infinite-recursion.cpp
+
----------------
$ is missing in the beginning.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100714