Icedtea-web and Java 9 again

Jiri Vanek jvanek at redhat.com
Tue Nov 3 13:08:04 UTC 2015


On 11/03/2015 11:27 AM, Jiri Vanek wrote:
> On 11/03/2015 10:17 AM, Fridrich Strba wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> Hello, good people,
>>
>> This change http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/2b680924a73f
>> broke again icedtea-web build with java 9. Since there is no support
>> of serialized applets anymore, the fix for Java 9 seems a straight
>> forward removal of the doInit = true assignment. See attached patch.
>>
>> Now there is a question, since the true is a default for doInit in the
>> base class, whether removing this doInit for java 7 and 8 is not a
>> plausible solution. If not, we will lose the possibility to use the
>> same build of icedtea-web for java 9 and the lower versions.
> I would like to keep this feature - build itw on one jdk, and run on another as  much as possible.
>
>>
>> Please, advise
>>
>
>
> Hello!
>
> Thanx for this! Really a lot. You are always kick to update jdk 9:)  I was playing with this issue,
> and I was not able to proof removal of the field is causing any harm. But eg I was not testing
> serialized appelts:-/
> And I assume, they will be the ones which break.
>
> Safest approach will be to handle this field doInit via  reflection.
> If it exists, set ti to true, otherwise ignore it.
>
> In this way are handeld eg. url permissions in ITW.
>
> I will try t reply with patch soon.

Attached is my "safer" patch. Thoughts? Will be valid for head and 1.6
>
> Thank you very much!
>    J.
>
>> Cheeers
>>
>> Fridrich
>> -----BEGIN PGP SIGNATURE-----
>> Version: GnuPG v2
>>
>> iEYEARECAAYFAlY4e5YACgkQu9a1imXPdA+BfgCfdDOLuGoEoOHqxEwLMo2+GLRz
>> uCoAn2Suv3wmTxIF68Rz3cKvnGucish+
>> =3N9E
>> -----END PGP SIGNATURE-----
>>
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: doInit-reflection.patch
Type: text/x-patch
Size: 2240 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20151103/0ca3f222/doInit-reflection.patch>


More information about the distro-pkg-dev mailing list