[PATCH] D72057: [docs] Remove outdated svn/git information from hacking page

Alexander Lanin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 2 16:50:33 PST 2020


AlexanderLanin updated this revision to Diff 235968.
AlexanderLanin retitled this revision from "[docs] Update dead anchor in hacking page" to "[docs] Remove outdated svn/git information from hacking page".
AlexanderLanin edited the summary of this revision.
AlexanderLanin added a comment.

Not sure we can call this updated proposal 'love': it's definitely not the goal to have less documentation, just less redundancy.

I'll remove redundancies between those two in a separate pull request:

- https://llvm.org/docs/Contributing.html#how-to-submit-a-patch
- https://llvm.org/docs/GettingStarted.html#sending-patches


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

https://reviews.llvm.org/D72057

Files:
  clang/www/hacking.html


Index: clang/www/hacking.html
===================================================================
--- clang/www/hacking.html
+++ clang/www/hacking.html
@@ -275,30 +275,8 @@
   <h2 id="patches">Creating Patch Files</h2>
   <!--=====================================================================-->
 
-  <p>To return changes to the Clang team, unless you have checkin
-  privileges, the preferred way is to send patch files
-  <a href="https://llvm.org/docs/Contributing.html#how-to-submit-a-patch">using LLVM's Phabricator</a> with an explanation of what the patch is for. Clang follows <a
-  href="https://llvm.org/docs/DeveloperPolicy.html">LLVM's developer policy</a>.
-  If your patch requires a wider discussion (for example, because it is an
-  architectural change), you can use the cfe-dev mailing list.</p>
-
-  <p>To create these patch files, change directory
-  to the llvm/tools/clang root and run:</p>
-
-  <pre>svn diff (relative path) >(patch file name)</pre>
-
-  <p>For example, for getting the diffs of all of clang:</p>
-
-  <pre>svn diff . >~/mypatchfile.patch</pre>
-
-  <p>For example, for getting the diffs of a single file:</p>
-
-  <pre>svn diff lib/Parse/ParseDeclCXX.cpp >~/ParseDeclCXX.patch</pre>
-
-  <p>Note that the paths embedded in the patch depend on where you run it,
-  so changing directory to the llvm/tools/clang directory is recommended.</p>
-
-  <p>It is also possible to <a href="https://llvm.org/docs/GettingStarted.html#sending-patches-with-git">use git to contribute</a> to Clang.</p>
+  <p>To contribute changes to Clang see
+    <a href="https://llvm.org/docs/GettingStarted.html#sending-patches">LLVM's Getting started page</a></p>
 
   <!--=====================================================================-->
   <h2 id="irgen">LLVM IR Generation</h2>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72057.235968.patch
Type: text/x-patch
Size: 1806 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200103/01602522/attachment-0001.bin>


More information about the cfe-commits mailing list