RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-11-25 12:12:47 +01:00
Code Issues Projects Releases Wiki Activity

[docs] Improve the documentation on committing code reviewed on

Browse Source 
Phabricator to trunk.

The previous documentation had a few issues:

* It did not make it explicit that code could be
committed without using the Arcanist tool and how this should be done.

* There was also an implicit assumption on using Subversion
rather than git-svn in the example using Arcanist. The documentation now
explicitly mentions both cases and details how to commit to trunk in
each case.

Reviewers: klimek, probinson

Subscribers: probinson, nwilson, reames, llvm-commits

Differential Revision: http://reviews.llvm.org/D15801

llvm-svn: 257764
This commit is contained in:
Dan Liew 2016-01-14 13:39:29 +00:00
parent 4c6d639e0b
commit 8f2866d9c0
2 changed files with 65 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
docs/GettingStarted.rst
View File

@ -628,6 +628,8 @@ Then, your .git/config should have [imap] sections.
; example for Traditional Chinese ; example for Traditional Chinese
folder = "[Gmail]/&g0l6Pw-" folder = "[Gmail]/&g0l6Pw-"
.. _developers-work-with-git-svn:
For developers to work with git-svn For developers to work with git-svn
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

83
docs/Phabricator.rst
View File

@ -127,18 +127,9 @@ a change from Phabricator.
Committing a change Committing a change
------------------- -------------------
Arcanist can manage the commit transparently. It will retrieve the description, Once a patch has been reviewed and approved on Phabricator it can then be
reviewers, the ``Differential Revision``, etc from the review and commit it to the repository. committed to trunk. There are multiple workflows to achieve this. Whichever
method you follow it is recommend that your commit message ends with the line:
::
arc patch D<Revision>
arc commit --revision D<Revision>
When committing a change that has been reviewed using
Phabricator, the convention is for the commit message to end with the
line:
:: ::
@ -147,17 +138,69 @@ line:
where ``<URL>`` is the URL for the code review, starting with where ``<URL>`` is the URL for the code review, starting with
``http://reviews.llvm.org/``. ``http://reviews.llvm.org/``.
Note that Arcanist will add this automatically.
This allows people reading the version history to see the review for This allows people reading the version history to see the review for
context. This also allows Phabricator to detect the commit, close the context. This also allows Phabricator to detect the commit, close the
review, and add a link from the review to the commit. review, and add a link from the review to the commit.
If you use ``git`` or ``svn`` to commit the change and forget to add the line Note that if you use the Arcanist tool the ``Differential Revision`` line will
to your commit message, you should close the review manually. In the web UI, be added automatically. If you don't want to use Arcanist, you can add the
under "Leap Into Action" put the SVN revision number in the Comment, set the ``Differential Revision`` line (as the last line) to the commit message
Action to "Close Revision" and click Submit. Note the review must have been yourself.
Accepted first.
Using the Arcanist tool can simplify the process of committing reviewed code
as it will retrieve reviewers, the ``Differential Revision``, etc from the review
and place it in the commit message. Several methods of using Arcanist to commit
code are given below. If you do not wish to use Arcanist then simply commit
the reviewed patch as you would normally.
Note that if you commit the change without using Arcanist and forget to add the
``Differential Revision`` line to your commit message then it is recommended
that you close the review manually. In the web UI, under "Leap Into Action" put
the SVN revision number in the Comment, set the Action to "Close Revision" and
click Submit. Note the review must have been Accepted first.
Subversion and Arcanist
^^^^^^^^^^^^^^^^^^^^^^^
On a clean Subversion working copy run the following (where ``<Revision>`` is
the Phabricator review number):
::
arc patch D<Revision>
arc commit --revision D<Revision>
The first command will take the latest version of the reviewed patch and apply it to the working
copy. The second command will commit this revision to trunk.
git-svn and Arcanist
^^^^^^^^^^^^^^^^^^^^
This presumes that the git repository has been configured as described in :ref:`developers-work-with-git-svn`.
On a clean Git repository on an up to date ``master`` branch run the
following (where ``<Revision>`` is the Phabricator review number):
::
arc patch D<Revision>
This will create a new branch called ``arcpatch-D<Revision>`` based on the
current ``master`` and will create a commit corresponding to ``D<Revision>`` with a
commit message derived from information in the Phabricator review.
Check you are happy with the commit message and amend it if necessary. Now switch to
the ``master`` branch and add the new commit to it and commit it to trunk. This
can be done by running the following:
::
git checkout master
git merge --ff-only arcpatch-D<Revision>
git svn dcommit
Abandoning a change Abandoning a change
------------------- -------------------