Request for review (XS): 7082220: Visual Studio projects broken after change 7016797: Hotspot: securely/restrictive load dlls and new

Bengt Rutisson bengt.rutisson at oracle.com
Tue Aug 23 09:26:35 UTC 2011


Hi all,

Could I please have a couple of reviews for this small fix? After the 
secure dll loading fix (7016797) the psapi.lib library is needed for 
Windows builds. The original change made sure that this library is 
provided to the linker for the command line builds. However, the builds 
from inside Visual Studio also need to know about this library.

Webrev:
http://cr.openjdk.java.net/~brutisso/7082220/webrev/

CR:
http://monaco.us.oracle.com/detail.jsf?cr=7082220

Testing:
I created a Visual Studio project with the create script and with the 
change above the project builds nicely.

I am including both Runtime and GC in this mail. The change is to 
runtime code, but I would like to push this through hotspot-gc. The 
reason is that this is blocking my work. Whenever I am setting up a new 
repository I run into this issue. But it seems that I am the only one 
who uses the Visual Studio builds at the moment. The issue has been 
around for several months but it only got integrated into hsx/hotspot-gc 
5 days ago. So, for me it would be easier to integrate directly into 
hotspot-gc and start using the fix rather than having to wait for the 
fix to propagate from hotspot-rt to hotspot-gc.

Thanks,
Bengt



More information about the hotspot-gc-dev mailing list