Review Request 112081: Add reviewboard-am, a tool to apply patches from KDE reviewboard

[Aurélien Gâteau]

> On Aug. 15, 2013, 12:55 p.m., Heinz Wiesinger wrote:
> > How does this compare to "rbt patch"? (http://www.reviewboard.org/docs/rbtools/0.5/rbt/commands/patch/)
> > 
> > It sounds to me like it's meant to accomplish the same thing
> 
> David Edmundson wrote:
>     That applies a patch to some code.
>     
>     Aurelien's version also commits it, correctly setting the commit author and filling out the commit message.
>

I thought I could switch to "rbt patch" when it was released, but as David says, "rbt patch" does not do the commit, so it's not as handy (and adding commit support to rbt is going to be complicated because rbt needs to support many VCS, most of which I have no knowledge of)


- Aurélien


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112081/#review37834
-----------------------------------------------------------


On Aug. 14, 2013, 5:26 p.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112081/
> -----------------------------------------------------------
> 
> (Updated Aug. 14, 2013, 5:26 p.m.)
> 
> 
> Review request for kdelibs and David Edmundson.
> 
> 
> Description
> -------
> 
> Add reviewboard-am, a tool to apply patches from KDE reviewboard.
> 
> (subscribing the "kdelibs" group to this request because there seems to be no "kde-dev-scripts" group)
> 
> 
> Diffs
> -----
> 
>   reviewboard-am PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/112081/diff/
> 
> 
> Testing
> -------
> 
> I and a few others, most notably David Edmundson, have been using it for quite some time now.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130816/f098b90c/attachment.htm>