No problem. I made a slight modification to your fix to replace the fixed 
password with a randomly generated one.
Here’s the webrev:
  http://cr.openjdk.java.net/~vinnie/8062264/webrev.00/

If you’re OK with that then I’ll push your fix along with two other fixes to 
KeychainStore.


BTW OCA approval is required before your patch can be integrated and I don’t 
see your name listed at:
  http://www.oracle.com/technetwork/community/oca-486395.html

Did you receive a confirmation email?


On 28 Oct 2014, at 09:02, Florian Bruckner (3kraft) 
<florian.bruck...@3kraft.com> wrote:

> Hi Vincent,
> 
> don't want to be pushy - just a gentle reminder :-)
> 
> regards,
> 
> Florian
> 
> On 14.10.14 15:13, Vincent Ryan wrote:
>> I’m the blocker. I’ll try to get your changset pushed by the end of this 
>> week.
>> 
>> 
>> On 14 Oct 2014, at 13:24, Florian Bruckner (3kraft) 
>> <florian.bruck...@3kraft.com> wrote:
>> 
>>> Hi Vincent,
>>> 
>>> it seems we've stalled on this somewhere.
>>> 
>>> What do I need to do to have the patch accepted?
>>> 
>>> regards,
>>> 
>>> Florian
>>> 
>>> 
>>> On 21.08.14 11:28, Vincent Ryan wrote:
>>>> Hello Florian,
>>>> 
>>>> I can sponsor this changeset and attribute the fix to you.
>>>> 
>>>> 
>>>> On 21 Aug 2014, at 09:52, Florian Bruckner (3kraft) 
>>>> <florian.bruck...@3kraft.com> wrote:
>>>> 
>>>>> Hi Sean,
>>>>> 
>>>>> the OCA has been approved - what are the next steps?
>>>>> 
>>>>> With best regards,
>>>>> 
>>>>> Florian
>>>>> 
>>>>> On 05.08.14 21:47, Sean Mullan wrote:
>>>>>> This seems like a reasonable change to me. In order to proceed with 
>>>>>> accepting your patch, you will first need to sign an OCA. See step 0 of 
>>>>>> http://openjdk.java.net/contribute/
>>>>>> 
>>>>>> Thanks,
>>>>>> Sean
>> 

Reply via email to