RFR: JDK-8212091 : Move native code under platform specific folders and files

Alexander Matveev alexander.matveev at oracle.com
Sat Feb 16 03:03:00 UTC 2019


Hi Magnus,

http://cr.openjdk.java.net/~almatvee/8212091/webrev.01/

Moved all files from "posix" to "unix" folder and reverted 
Lib-jdk.jpackage.gmk changes.
Webrev updated with files moved, instead of add/remove.

Thanks,
Alexander

On 2/14/2019 11:44 PM, Magnus Ihse Bursie wrote:
>
>
> On 2019-02-15 04:31, Alexander Matveev wrote:
>> Please review the jpackage fix for bug [1] at [2].
>>
>> This is a fix for the JDK-8200758-branch branch of the open sandbox 
>> repository (jpackage).
>>
>> - Moved native code under platform specific folder.
>> - Removed most usage on #ifdefs for WINDOWS, LINUX, MAC and POSIX.
>> - MAC define is still used in JavaVirtualMachine.cpp and Package.cpp 
>> for Mac specific code to filter out some arguments. I decided to keep 
>> it as is for now, since Mac specific code is small.
>> - Defines are used in Platform.cpp to initialize platform specific 
>> classes.
>> - Removed all pragma warning and fixed all compilation warnings.
>> - Removed unused code.
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8212091
>>
>> [2] http://cr.openjdk.java.net/~almatvee/8212091/webrev.00/
> The JDK standard is to use "unix", not "posix", for the shared 
> functionality between linux/solaris/macosx. You can keep the name 
> "PosixPlatform.*" if you want, though; the important thing is the 
> directory name.
>
> Also, if you do that, you do not need any changes to 
> make/lib/Lib-jdk.jpackage.gmk, since that will be automatically 
> understood by the build system.
>
> It looks from the webrev that you have "moved" the files by doing "hg 
> add" and "hg remove". Please use "hg move" instead -- this will keep 
> history intact, and it allows reviewers to see if you have also made 
> changes to the moved files.
>
> (If you do have modified the moved file, reverting a "hg add+hg 
> remove" process is a bit more tricky -- you need to do "hg forget" on 
> the new file, rename it to something else (otherwise "hg move" will 
> complain), "hg revert" the old file back in place, do a "hg move" from 
> the old to the new, and then copy the modified, renamed file back over 
> the target new file again.)
>
> /Magnus
>>
>> Thanks,
>> Alexander
>




More information about the build-dev mailing list