search for: d1234

...a > patches-based approach with small changes. For me that corresponds to > one commit per code review. When I address code review feedback in a > parent revision I use git's interactive rebase. It's worth mentioning that Phabricator can read strings of the format 'Depends on D1234' from commit messages and create those relationships for you. Also just because it's not easy to find unless you know it's there. You can view the parent/child relationships in the 'Revision Contents' section on the 'Stack' tab. >> The interface for reviewing and...

Personally I'd like a "git llvm fetch-diff D1234" subcommand, which outputs a patch in the format of "git format-patch", which, you could then pipe to "git am", say, providing whatever prefix options you like. On Fri, Feb 8, 2019 at 2:41 PM Mehdi AMINI via llvm-dev < llvm-dev at lists.llvm.org> wrote: > Hi, &gt...

...020 at 11:32 AM Renato Golin via llvm-dev < llvm-dev at lists.llvm.org> wrote: > On Wed, 8 Jan 2020 at 02:26, Daniel Sanders via llvm-dev > <llvm-dev at lists.llvm.org> wrote: > > It's worth mentioning that Phabricator can read strings of the format > 'Depends on D1234' from commit messages and create those relationships for > you. > > Side note: I find that there's too many unknown features of Phab that > require manual annotations on the commit message to work. > > I don't think that's a good strategy. New contributors get lost...

Hi all, The Phabricator docs seem to say that downloading a patch goes with `arc patch`. However, I tried it on several revisions and they all fail with: Checking patch some/path/to/a/file/that/exists ... error: some/path/to/a/file/that/exists: does not exist in index With the svn repo I found that command to be really useful. But now it seems like I have to download the patch manually, create

...d approach with small changes. For me that corresponds to > > one commit per code review. When I address code review feedback in a > > parent revision I use git's interactive rebase. > > It's worth mentioning that Phabricator can read strings of the format 'Depends on D1234' from commit messages and create those relationships for you. > > Also just because it's not easy to find unless you know it's there. You can view the parent/child relationships in the 'Revision Contents' section on the 'Stack' tab. > > >> The interfa...

On Tue, Jan 7, 2020 at 4:59 PM Doerfert, Johannes <jdoerfert at anl.gov> wrote: > Hi Bill, > > On 01/07, Bill Wendling via llvm-dev wrote: > > Then perhaps those opposed could suggest how to use Phabricator/Arcanist > so > > that I don't throw my keyboard through my monitor? > > Please explain your problems, w/o the hyperbole, so people can actually do >

...^ ------------------------------------------------------------------------ c89 -I. -I../../src/include -I../../src/include -I/usr/local/include -DHAVE_CONFIG_H -OPT:IEEE_NaN_inf=ON -g -c saveload.c -o saveload.o cc-1552 c89: WARNING File = saveload.c, Line = 1600 The variable "d1234" is set but never used. int d1, d2, d3, d4, d1234; ^ cc-1552 c89: WARNING File = saveload.c, Line = 44 The variable "NVSize" is set but never used. static int NVSize; /* Number of vector cells */ ^ ----------------------...