> 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
_______________________________________________
Patchwork mailing list
[email protected]
https://lists.ozlabs.org/listinfo/patchwork

Reply via email to