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

Pete Brunet peter.brunet at oracle.com
Thu Oct 27 23:50:47 UTC 2016



On 10/27/16 6:31 PM, Pete Brunet wrote:
> On 10/27/16 1:30 PM, Mandy Chung wrote:
>>> On Oct 27, 2016, at 10:44 AM, Phil Race <philip.race at oracle.com> wrote:
>>>
>>> No, we are definitely shipping those.
>>> Unless of course you think we should stop shipping JNI headers too …
>>>
>> No.  I tried to understand what is external interface.  I took it that these header files are external interfaces.
>>
>> I reviewed:
>>   http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.05/index.html
>>
>> Please move AccessBridgeCalls.c to  src/jdk.accessibility/windows/native/bridge directory.
> I moved the source to
> src/jdk.accessibility/windows/native/bridge/src
> and the includes to
> src/jdk.accessibility/windows/native/bridge/include
>
> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.06/
I forgot to update the link in readme.html, i.e. include/bridge is now
bridge/include.  See
http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.07/
>> Right now, the header files to be packaged in the JMOD file and linked into the image is white-listed in the makefile.
>>
>> Ultimately, we want the entire directory to contain only header files for external interfaces and then the build can simply take all files under src/${MODULE}/share,${OS}/native/include and get rid of the hardcoded list in the makefile.  We are not there yet - one open issue is related to the duplicated header files in VM and JDK and some are private and some are external.  But we should prepare for that.
>>
>> Mandy
>>
>>




More information about the build-dev mailing list