This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH,HURD] hurd: compliance fixes for getgroups
- From: Roland McGrath <roland at hack dot frob dot com>
- To: Pino Toscano <toscano dot pino at tiscali dot it>
- Cc: bug-hurd at gnu dot org, libc-alpha at sourceware dot org
- Date: Mon, 14 May 2012 17:01:13 -0700 (PDT)
- Subject: Re: [PATCH,HURD] hurd: compliance fixes for getgroups
- References: <201204271243.52809.toscano.pino@tiscali.it><20120427215402.9771E2C0AF@topped-with-meat.com><201204281322.02499.toscano.pino@tiscali.it>
> > Since I see you already have
> > copyright paperwork and you're doing more than just the one change,
> > would you like to get set up to commit yourself?
>
> Given also my contributions so far, I guess that I should better need
> review for now, no?
Everybody needs review. Getting set up to do your own commits doesn't mean
you do them without approval, just that you can do the mechanical part
yourself instead of reviewers having to do it for you. See the wiki.
This change looks fine to me. Please follow the wiki instructions to get
set up to commit it yourself. To make sure you're doing it right, don't
push to "master" but push it to a "<yourname>/<something>" private branch
as the wiki explains. Then I'll double-check that you did everything right
and merge the branch in. After that when I tell you a change is approved,
you can just push it directly to "master".
Thanks,
Roland