[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Rietveld workflow problems
From: |
David Kastrup |
Subject: |
Re: Rietveld workflow problems |
Date: |
Wed, 21 Sep 2011 16:00:21 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux) |
Reinhold Kainhofer <address@hidden> writes:
> Am Wednesday, 21. September 2011, 15:04:05 schrieb David Kastrup:
>> Reinhold Kainhofer <address@hidden> writes:
>> > Am Wednesday, 21. September 2011, 10:52:37 schrieb David Kastrup:
>> >> Perhaps it would be nice if we found a way to play with "Gerrit",
>> >> supposedly a git-based system similar to Rietveld.
>> >
>> > I looked at gerrit a while ago. If you want to take a look at it:
>> > http://server.kainhofer.com:8088/
>> >
>> > Here is a quick summary:
>> >
>> > -) Each review is basically one commit in a branch on the gerrit server.
>>
>> Well, that's pretty much what we already have.
>
> No. In gerrit you really need to clean up your patches before you
> submit them for review. I typically have lots of small commits in a
> branch when I upload a patch to rietveld. git-cl will simply take the
> diff to origin/master (i.e. all patches in the branch combined into
> one large patch) and show the combined diff. gerrit, on the other
> hand, will show each small patch as one review.
So what?
Assume you have committed everything on branch untidy. Then you do
git rebase origin
git checkout -b tidy origin
git checkout -p untidy
And voila, you have patched in the current state of the untidy HEAD into
your brandnew tidy branch and may choose to commit it.
> So, I would have to clean up my local branch before submitting for
> review (i.e. rebase -i and squash and reorder the patches). That's a
> very fundamental difference in the approach. Rietveld works on the
> diff level, gerrit on the git commit level.
It is not hard to munch everything into one commit. The complex thing
is dealing sensibly with stuff that you do _not_ want to have munched
into a single commit.
--
David Kastrup
- Rietveld workflow problems, David Kastrup, 2011/09/21
- Re: Rietveld workflow problems, address@hidden, 2011/09/21
- Re: Rietveld workflow problems, David Kastrup, 2011/09/21
- Re: Rietveld workflow problems, Reinhold Kainhofer, 2011/09/21
- Re: Rietveld workflow problems, David Kastrup, 2011/09/21
- Re: Rietveld workflow problems, Reinhold Kainhofer, 2011/09/21
- Re: Rietveld workflow problems, Janek Warchoł, 2011/09/21
- Re: Rietveld workflow problems,
David Kastrup <=
- Re: Rietveld workflow problems, Janek Warchoł, 2011/09/21
- Re: Rietveld workflow problems, Carl Sorensen, 2011/09/21
- Re: Rietveld workflow problems, Graham Percival, 2011/09/21
- Re: Rietveld workflow problems, Janek Warchoł, 2011/09/22
- Re: Rietveld workflow problems, Carl Sorensen, 2011/09/22
Re: Rietveld workflow problems, Peekay Ex, 2011/09/21
Re: Rietveld workflow problems, Reinhold Kainhofer, 2011/09/21