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

Jesper Wilhelmsson jesper.wilhelmsson at
Tue Aug 23 02:46:19 PDT 2011

Ship it!

On 08/23/2011 11:26 AM, Bengt Rutisson wrote:
> 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:
> CR:
> 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-runtime-dev mailing list