[OE-core] [daisy][PATCH 1/3] layer.conf: Add in useradd dependencies as ABISAFE dependencies

Richard Purdie richard.purdie at linuxfoundation.org
Wed Oct 1 08:12:16 UTC 2014


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 <denis at 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 <denis at 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 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 :(.

Cheers,

Richard









More information about the Openembedded-core mailing list