(original) (raw)

Hi,

Are you trying to apply a revision that was uploaded to Phabricator from one of the individual repository (or from SVN)?

The issue if I remember correctly is that a patch uploaded from an SVN checkout would not start with \`llvm/\` or \`clang/\` and wouldn't be able to be applied in the monorepo. Conversely, a revision created from the monorepo can't be applied by \`arc patch\` in a single subproject checkout.

I don't think there is a way around this, maybe \`--strip-prefix\` and \`--add-prefix\` options could be added to \`arc patch\`, but I'm not sure it would fit the expectation of \`arc\` upstream.

Mehdi

On Fri, Feb 8, 2019 at 10:42 AM Nicolas Lesser via llvm-dev <llvm-dev@lists.llvm.org> wrote:

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 a new branch, apply it and create a commit for it. Having \`arc patch\` would be a really great improvement to that - since it also does arc specific stuff I believe that I can't do manually (like arc diff telling me which patch to commit).

Is there a way I can make it work?

Thanks
P.S: I'm not on llvm-dev list, so any replies would have to mention my email I guess :)
- Nicolas
\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_
LLVM Developers mailing list
llvm-dev@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev