Changes between Version 8 and Version 9 of Phabricator


Ignore:
Timestamp:
Mar 19, 2019, 1:17:55 AM (5 years ago)
Author:
s0600204
Comment:

Clarify a point, and set syntax highlighting for code/shell extracts.

Legend:

Unmodified
Added
Removed
Modified
  • Phabricator

    v8 v9  
    4444Take a look at the [wiki:Phabricator#CommonproblemswithArcanist Common problems with Arcanist] section below if you have any trouble.
    4545
    46 Finally, associate arc with your credentials on code.wildfiregames.com using
    47 {{{
     46Finally, associate arc with your credentials with our Phabricator instance by using
     47{{{#!sh
    4848arc install-certificate
    4949}}}
     
    5151Two commands are important, that you can run at the root of a SVN clone (where .arcconfig is):
    5252
    53 {{{
     53{{{#!sh
    5454arc patch Dn
    5555}}}
    5656applies the revision Dn (n being an integer) to your working copy. No need to download and apply, easy to update when a contributor updates a revision with a new diff, and more...
    5757
    58 {{{
     58{{{#!sh
    5959arc diff --preview
    6060}}}
     
    7070
    7171It is recommended to generate the patch with full file context so review is eased on Phabricator. Use
    72 {{{
     72{{{#!sh
    7373svn diff --diff-cmd=diff -x -U999999
    7474}}}
     
    7878
    7979If you have commit access, you can clone the non-public repository. Then, you can use
    80 {{{
     80{{{#!sh
    8181arc patch Dn
    8282svn st
     
    8585
    8686The commit message must contain
    87 {{{
     87{{{#!sh
    8888Differential Revision: https://code.wildfiregames.com/Dn
    8989}}}
     
    9191
    9292A useful command is
    93 {{{
     93{{{#!sh
    9494arc commit --revision Dn --show
    9595}}}
     
    103103
    104104To do so, I suggest running:
    105 {{{
     105{{{#!sh
    106106svn patch /path/to/your_patch.diff
    107107arc diff --preview
     
    121121* Differential revisions will be created without context ([https://secure.phabricator.com/T2465 T2465]). In order to work around the issue, you can change the behavior of `svn diff` the following way:
    122122 * Create a file called `contextdiff.bat` with the following contents:
    123 {{{
     123{{{#!bat
    124124@echo off
    125125diff.exe -U9999999 -L %3 -L %5 %6 %7
     
    127127  This file will use the `diff` executable provided by the Git Bash environment, so it should work.
    128128 * Change you SVN settings: go to `%appdata%\Subversion\` and edit the `config` file. Find the line beginning with
    129 {{{
     129{{{#!bat
    130130# diff-cmd = ...
    131131}}}
    132132  and replace it by
    133 {{{
     133{{{#!bat
    134134diff-cmd = X:\full\path\to\contextdiff.bat
    135135}}}
    136136 * You can test it works by running `svn diff` in the repository: the output should contain the entire contents of the modified file. From now on `arc diff` will create diffs with context available.
    137137* You will need to patch Arcanist against [https://secure.phabricator.com/T8075 T8075]. To do so, in your clone of the arcanist git repository, run
    138 {{{
     138{{{#!sh
    139139git remote add oujesky https://github.com/oujesky/arcanist.git
    140140git fetch oujesky
     
    145145
    146146If Arcanist "cannot parse svninfo" it's because your locale outputs the result of the command "svn info" in something that is not English. You can write to your shell rc file something like
    147 {{{
     147{{{#!sh
    148148alias arc='LANG=C LC_ALL=C arc'
    149149}}}