Hi Omair,

Omair Majid <oma...@redhat.com> wrote:

>Hi Alex,
>
>* Alex Kasko <m...@alexkasko.com> [2014-05-24 13:20]:
>> I've built 32-bit and 64-bit versions successfully using Windows SDK
>7.1
>> (without Visual Studio) with the following patch:
>> 
>> $ diff jdk6_vs2010/jdk/src/windows/resource/version.rc version.rc
>> 26c26
>> < #include "afxres.h"
>> ---
>> > #include "windows.h"
>> 
>> $ diff jdk6_vs2010/jdk/src/windows/native/sun/windows/awt.rc awt.rc
>> 26c26
>> < #include "afxres.h"
>> ---
>> > #include "windows.h"
>> 40,42c40,42
>> < SECURITY_WARNING_0  ICON    DISCARDABLE "security_warning_bw.ico"
>> < SECURITY_WARNING_1  ICON    DISCARDABLE "security_warning_int.ico"
>> < SECURITY_WARNING_2  ICON    DISCARDABLE "security_warning.ico"
>> ---
>> > SECURITY_WARNING_0  ICON    DISCARDABLE "awt.ico"
>> > SECURITY_WARNING_1  ICON    DISCARDABLE "awt.ico"
>> > SECURITY_WARNING_2  ICON    DISCARDABLE "awt.ico"
>
>I have been told that this patch is still needed on Windows. Do you
>have
>commit access to OpenJDK6? If so, please file a bug, count this as a
>"review" and push the change. If not, can you create a changeset? I can
>push it for you.
>
>Thanks,
>Omair
>
>-- 
>PGP Key: 66484681 (http://pgp.mit.edu/)
>Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681


I do not have commit access. I am not sure about bugtracker (I have access to 
JBS, but never tried JDK6 tracker), I'll try it.

I'll create the changeset, need some days though to test patch against current 
tip and to check why the icons change is required - windows.h header change is 
straightforward, but I do not understand the reason of missing icons.

-- 
-Alex

Reply via email to