(original) (raw)

Hi Aleksey,

Thanks for fixing this in OpenJDK!
I actually noticed the same issue a few weeks ago \[1\], but somehow I had missed the reply that Roland sent me, so I didn't send out a request for review for my version of the change.
But now that it's fixed, everythings all right ;-)

Thanks,
Kris

\[1\]: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2015-June/018179.html

On Mon, Jul 27, 2015 at 1:51 AM, Aleksey Shipilev <aleksey.shipilev@oracle.com> wrote:
Thanks for pushing the change, Dean!

\-Aleksey

On 07/25/2015 05:32 AM, John Rose wrote:
\> You are fine.
\> Aleksey is an Author for JDK 9: http://openjdk.java.net/census#shade
> Committers who sponsor changes are expected to use the correct Author
\> (not themselves).
\> It's a syntax error to push a non-Author changeset to an OpenJDK repo.
\> BTW, jcheck does not consult the OJN census AFAIK.
\>
\> — John
\>
> On Jul 24, 2015, at 6:35 PM, Dean Long <dean.long@oracle.com
> dean.long@oracle.com>> wrote:
\>>
\>> OK I will push it now. I did 'hg ci -u shade' so that Aleksey gets
\>> credit for it, and jcheck isn't complaining. Does anyone know if JPRT
\>> does more checks for Committer status? If so I'll have to redo it and
\>> add a Contributed-by line.
\>