[11u] RFR: jnf_removal, series of patches
Vladimir Kempik
vkempik at azul.com
Fri Apr 16 12:43:50 UTC 2021
Tier2 testing is good as well.
> 16 апр. 2021 г., в 15:12, Vladimir Kempik <vkempik at azul.com> написал(а):
>
> The collated webrev:
> http://cr.openjdk.java.net/~vkempik/jnf_removal/webrev.01/ <http://cr.openjdk.java.net/~vkempik/jnf_removal/webrev.01/>
>
> Testing - tier1, will soon update on tier2.
>
>> 16 апр. 2021 г., в 14:39, Vladimir Kempik <vkempik at azul.com> написал(а):
>>
>> Hello
>> Please review this backport of 11 patches from jdk17 to jdk11, they remove dependendy on JavaNativeFoundation.framework on macos
>> It’s 7 original patches fo jnf removal and 4 postmortem items to fix bugs.
>> I have created one collated webrev for 11 changesets, please let me know if it’s not the best way to do it.
>> Brief summary for backports:
>> 1_8257858 & 8257860 - Applies almost clean, few gmk files had different filename and few copyright year issues, context difference in gmk files.
>> 2_8257853 - Not clean, most of issues were in CPrinterJob and JavaAccessibility due to changes in jdk12+. Rest was mostly context code difference.
>> 3_8259343 - Clean, only gmk files renames were needed
>> 4_8259651 - Not clean, have to remove some changes to parts which are missing in jdk11: CMenuBar: ActivateDefaultMenuBar, Accessebility: titleChanged, getMultiClickTime
>> 5_8259869 - Almost Clean, slight Context code difference in CGLLayer.h
>> 6_8260616 & 8259729 - not clean, Missing CommonTextAccessibility.m in jdk11. Some Context code difference. Two changes in one patch to not break the build
>> 7_8257988 - Clean
>> 8_8259232 - clean
>> 9_8259585 - clean
>> 10_8261198 - Clean
>> 11_8263846 - clean
>>
>> So I need review for patches #1 #2 #4 #5 #6 and maybe (not sure) #3
>>
>> #1 the bug - https://bugs.openjdk.java.net/browse/JDK-8257858
>> Original changeset - https://github.com/openjdk/jdk/commit/4a8b5c16
>>
>> #2 the bug - https://bugs.openjdk.java.net/browse/JDK-8257853
>> Original changeset - https://github.com/openjdk/jdk/commit/fa50877c
>>
>> #3 the bug - https://bugs.openjdk.java.net/browse/JDK-8259343
>> Original changeset - https://git.openjdk.java.net/jdk/commit/d6a2105b
>>
>> #4 the bug - https://bugs.openjdk.java.net/browse/JDK-8259651
>> Original changeset - https://github.com/openjdk/jdk/commit/5855d52a
>>
>> #5 the bug - https://bugs.openjdk.java.net/browse/JDK-8259869
>> Original changeset - https://github.com/openjdk/jdk/commit/92c2f084
>>
>> #6 the bug - https://bugs.openjdk.java.net/browse/JDK-8260616
>> Original changeset - https://github.com/openjdk/jdk/commit/8760688d
>>
>> Regards, Vladimir
>>
>
More information about the jdk-updates-dev
mailing list