[llvm-dev] Proposing a llvm-patch helper script in-tree to create/apply patches without arc

Martin Storsjö via llvm-dev llvm-dev at lists.llvm.org
Mon Jan 20 22:44:38 PST 2020


On Tue, 21 Jan 2020, David Tellenbach via llvm-dev wrote:

> The downsides of providing a custom script are obvious: Someone has to 
> write it and someone has to maintain it. Moreover I don't think that 
> arcanist is so complicated that a custom script would be a huge 
> improvement (you'd have to read the documentation of the new tool 
> anyway).

To me, one upside would be if it simply would do less; if I want to apply 
a patch using arcanist, it will automatically change branches to apply the 
patch on the right base revision (or maybe just fetch the very latest 
master and apply it on top of that). As the llvm-project repo is fairly 
large, rebuilds are costly, and updating the git tree to a new version is 
something I want to schedule myself, not have a tool do for me behind my 
back when I want to try out a patch (that most probably would work just 
fine on my couple-days-old tree).

I would much prefer a tool that simply tries to apply the change on the 
current branch that I happen to have checked out at the moment - 
essentially "git am". I've tried to look for arcanist flags to achieve 
this, but didn't find any when I looked.

So a simple standalone tool (without the php dependency) that does less 
automatic stuff and just downloads and applies the patch would be great to 
me. Bonus points if it would set the git author field properly based on 
the patch author's phabricator user name/mail address.

// Martin



More information about the llvm-dev mailing list