RfR JDK-8167213 Move include/bridge/AccessBridgeCalls.c to the source directory

Phil Race philip.race at oracle.com
Mon Oct 31 16:38:30 UTC 2016


+1.

I am assuming you made sure AccessBridgeCalls.c is not being compiled
during the JDK build as discussed earlier ...

-phil.

On 10/31/2016 07:36 AM, Pete Brunet wrote:
>
> On 10/28/16 8:14 PM, Mandy Chung wrote:
>>> On Oct 28, 2016, at 1:59 PM, Philip Race <philip.race at oracle.com> wrote:
>>>
>>> If it is not in the image then there is no point in the file existing.
>>> Maybe this could just be a comment at the top of the include file.
>>>
>> This works for me.
> Updated:
> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.09/
>> Mandy
>>
>>> -phil.
>>>
>>> On 10/28/16, 12:42 PM, Mandy Chung wrote:
>>>>> On Oct 28, 2016, at 11:32 AM, Pete Brunet<peter.brunet at oracle.com>  wrote:
>>>>>
>>>>> Hi Mandy, That simplifies things.  The new patch is at:
>>>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.08/
>>>> Looks better.
>>>>
>>>> I only notice now that the readme.html is in the include directory.  That should be in the documentation as you proposed earlier.  I don’t think it should be copied to the image.
>>>>
>>>> Mandy
>>>>




More information about the build-dev mailing list