Re: Code reviews

[Phillip Lord]

Eli Zaretskii <address@hidden> writes:

>> From: address@hidden (Phillip Lord)
>> Date: Tue, 08 Mar 2016 08:48:56 +0000
>> Cc: address@hidden
>> 
>> Giving people different permissions on different branches seems to make
>> sense. Overtime, people move toward master/emacs-25.
>
> AFAIK, Git (or is it Savannah?) doesn't support such granularity.

Yes, it does, through hooks. I think, most people just use gitolite for
the purpose.


>> > Maybe we could have a half-way system, where commits are pushed to
>> > a branch that is "not fast-forward-only", and this branch is then
>> > auto-merged to the real (fast-forward-only) master branch after a delay
>> > (one day, maybe?) to give time to fix mess ups before they're cast
>> > in stone.
>> 
>> I think that this would require some considerable co-ordination. If I
>> push a broken commit to devel branch, and then you fix this commit
>> through rebase, my copy of the branch (and everyone elses) is now
>> broken.
>
> Stefan was talking about merging, not rebasing.

I think he was talking about both. "fix mess ups" sounds like a rebase
to me. But I was confused about his overall workflow.

Phil