Hi Volker, 

the change looks good.  Thanks for fixing this.

Best regards,
  Goetz.

-----Original Message-----
From: serviceability-dev [mailto:serviceability-dev-boun...@openjdk.java.net] 
On Behalf Of Volker Simonis
Sent: Mittwoch, 6. Mai 2015 18:12
To: jdk9-...@openjdk.java.net; serviceability-dev@openjdk.java.net
Subject: RFR(XS): 8079510: AIX: fix build after '8042901: Allow 
com.sun.management to be in a different module...'

Hi,

can somebody please review this trivial AIX-only change which fixes an
AIX build error:

http://cr.openjdk.java.net/~simonis/webrevs/2015/8079510/
https://bugs.openjdk.java.net/browse/JDK-8079510

Here are the details:

Change '8042901: Allow com.sun.management to be in a different module
to java.lang.management' has refactored the management package and
moved the files:

jdk/src/java.management/unix/native/libmanagement/<OS>OperatingSystem.c

to:

jdk/src/jdk.management/<OS>/native/libmanagement_ext/UnixOperatingSystem.c

Unfortunately we've never had a AIXOperatingSystem.c file on AIX (see
8030957). This was no problem with the old setup but with the new one
the build complains about a missing directory
'jdk/src/jdk.management/aix/native/libmanagement_ext'.

This bug is just for fixing the build, by introducing an emty stub
file. The actual implementation of the corresponding
OperatingSystemMXBean is still up to 8030957.

Thanks,
Volker

Reply via email to