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

[Jesper Wilhelmsson]

Ship it!
/Jesper

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:
> 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