Op 1 okt. 2014, om 10:12 heeft Richard Purdie 
<richard.pur...@linuxfoundation.org> het volgende geschreven:

> On Wed, 2014-10-01 at 06:10 +0200, Martin Jansa wrote:
>> On Tue, Sep 30, 2014 at 08:55:53PM -0700, Khem Raj wrote:
>>> On Tue, Sep 30, 2014 at 6:32 PM, Denys Dmytriyenko <de...@denix.org> wrote:
>>>> On Tue, Sep 30, 2014 at 06:25:57PM -0700, Khem Raj wrote:
>>>>> On Tue, Sep 30, 2014 at 5:55 PM, Denys Dmytriyenko <de...@denix.org> 
>>>>> wrote:
>>>>>> This line above ^^^^ has a very long trail of whitespaces and it got 
>>>>>> commited
>>>>>> that way, unfortunately :(
>>>>> 
>>>>> Better was if you commented before it went in.
>>>> 
>>>> And how would you suggest I was supposed to see it?? Nobody saw it. My 
>>>> email
>>>> client is no different than others - it doesn't mark trailing whitespaces. 
>>>> The
>>>> reason I saw it is because I ran "git log -p" on the file and that 
>>>> explicitly
>>>> highlights trailing whitespaces in BRIGHT RED... :)
>>> 
>>> just for that may be we should start using gerrit ;)
>> 
>> I'm all for it, it would make my e-mail - review - patchwork -
>> master-next workflow a lot easier and reliable.
> 
> And I am not. Why? I've actually tried using the thing and I absolutely
> *despise* its interface. This isn't just a mild dislike, I simply cannot
> do the things I want/need to do with it.

I use gerrit at work and I despise the interface as well, but it does address a 
number is issues plaguing OE-core:

1) pull-request doesn't match emailed patches
2) Patches are forgotten
3) Wrong version of patch gets applied
4) Review comments aren't tracked

But patchwork can be used to address the above problems as well. Anyway, this 
isn't a purely technical problem, see below.

> I believe I should have some kind of a say in this since it will
> directly affect my workflow and general life (since I spend a lot of my
> time dealing with this). Its appears to be very easy for other people to
> tell me how I should work though :(.

I think the real solution doesn't involve extra layers of tools, but a move to 
a kernel style environment where subsystem maintainers collect and vet patches 
and send pull requests to RP. But that would require formalizing the commit and 
review rules which I suspect will be controversial.

--
Koen
-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to