[PULL] github.com/stephenfin/development

[Finucane, Stephen]

> On Fri, Oct 16, 2015 at 03:17:42PM +0100, Finucane, Stephen wrote:
> > > On Thu, Oct 15, 2015 at 11:46:30PM +0000, Finucane, Stephen wrote:
> > > > Hey,
> > > >
> > > > First pull request (request-pull style, that is). Hope I've done
> > > > everything correctly :S This PR includes two fixes and the code
> enable
> > > > the check API. It's got positive reviews first time round and I
> > > > haven't heard any complaints for this revision so we're good to go.
> > >
> > > This conflicts with the pull request I sent two weeks ago, I'm afraid.
> >
> > There didn't seem to be any movement on that pull request so I figured I
> should push on. However, your series is there longer and therefore takes
> priority:
> >
> > 	https://lists.ozlabs.org/pipermail/patchwork/2015-October/001802.html
> >
> > I'll get that series merged first then rework this PR.
> >
> > Hope this is OK?
> 
> You are sending a pull request with unreviewed patches that introduce db
> schema changes and new APIs. I don't think we should do that. It's quite
> cheeky to present the work as ok because the last round didn't get any
> replies (but wasn't addressing the concerns either).
> 
> The elephant in the room is that the proposed db schema cannot handle
> the use case I want to cover, while my modification still handles the
> policy you'd like.

I addressed the feedback from Thomas Monjalon and Thomas F Herbert, who gave me meaningful feedback and stayed around to address my comments. You, on the other hand, will neither review my patches (either at all or a meaningful way[1]) nor reply to my comments[2][3], Damien. Please tell me how else you expect me to get things merged in this environment? You expressed frustration about the lack of replies to your own patches but what about me: I review and test almost every single patches you submit, regardless of how big they are [4], and yet you don't return the favour. Please be a good community partner and help me out here or I'll have no choice but to go around you.

Stephen

[1] https://lists.ozlabs.org/pipermail/patchwork/2015-October/001880.html
[1] https://lists.ozlabs.org/pipermail/patchwork/2015-September/001710.html
[2] http://wiki.qemu.org/Contribute/SubmitAPatch#Pay_attention_to_review_comments
[3] http://wiki.qemu.org/Contribute/SubmitAPatch#Make_code_motion_patches_easy_to_review