Review request: Bug #9632
abartlet at samba.org
Sun Jun 9 02:38:23 MDT 2013
On Sun, 2013-06-09 at 09:53 +0200, Kai Blin wrote:
> Hi folks,
> two weeks ago I fixed bug #9632 (DNS can't handle large packets), but I
> still need a review to get the patches into 4.0. Can someone please
> review so we can get this to our users pre-4.1?
I'm always happy to review patches. To get patches in bugzilla to my
review attention, please set the review flag in the detail on the patch
to ? and then my e-mail address.
I realise it feels like 'make-work', but please 'git cherry-pick -x' the
patches into a branch of 4.0, and then upload the git format-patch -1
output, as it provides us with a link between the patches, so if issues
are found with it later, we can follow the original context in master.
I know the process can feel painful, but this much seems to work OK as a
practice for getting patches into 4.0, and causes a mail to be generated
to me that I can quickly act on. Also, by asking specifically to my
attention I can see your review request in this query, to help me catch
up when I've been swamped:
And for your curiosity, I can see my own review requests with this one:
I hope this helps avoid your patches getting lost again, because I very
much value your contribution here.
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
More information about the samba-technical