some replacements for encumbered binary plugs no longer required?

[Lillian Angel]

Hi,

Lillian Angel wrote:
> 
> 
> Paul Jenner wrote:
>> Hi all.
>>
>> It looks like some of the replacements for encumbered binary plugs in
>> IcedTea are no longer required - as plugs have either been opened or
>> replaced in OpenJDK.
>>
>> As a quick first pass:
>>
>> 1) Ductus renderer:
>>
>> Looks like it was replaced in OpenJDK here:
>> http://mail.openjdk.java.net/pipermail/graphics-rasterizer-dev/2007-October/000049.html 
>>
>>
>> Allowing removal of:
>>
>> libdcpr.so (from Makefile.am)
>>
>> rt/sun/dc
>> rt/sun/dc/pr
>> rt/sun/dc/pr/PRException.java
>> rt/sun/dc/pr/PathDasher.java
>> rt/sun/dc/pr/Rasterizer.java
>> rt/sun/dc/pr/PathStroker.java
>> rt/sun/dc/path
>> rt/sun/dc/path/PathException.java
>> rt/sun/dc/path/PathConsumer.java
>> rt/sun/dc/path/FastPathProducer.java

These don't exist in the b23/b24 bundle. Possibly b25 or later?

>>
>> 2) Encumbered sound:
>>
>> Looks like it was partially opened here:
>> http://mail.openjdk.java.net/pipermail/sound-dev/2007-September/000037.html 
>>
>>
>> Allowing removal of:
>>
>> libjsoundalsa.so (from Makefile.am)
>> libjsound.so (from Makefile.am)
>>
>> (only libjsoundhs.so looks to be needed from the referenced e-mail)

I think you are correct here.

>>
>> 3) Lib2k font library:
>>
>> Looks like it was replaced in OpenJDK here:
>> http://mail.openjdk.java.net/pipermail/2d-dev/2007-August/000037.html
>>
>> Allowing removal of:
>>
>> libt2k.so (from Makefile.am)

Right.

>>
>> 4) unused gnu. imports:
>>
>> It looks like:
>>
>> rt/gnu/java/io
>> rt/gnu/java/io/Base64InputStream.java
>>
>> was left over from the merge of GNU crypto and is no longer used.

Oops.

>>
>> I intended to try a build with these removed to confirm they really
>> weren't required but hit a problem with ecj build (org.xml.sax.helpers
>> not found) so thought I would mail for comment in the meantime.
>>
>> Is there any intention to review and keep the replacement plugs as small
>> as possible - or is it simply not worth the effort of removal now as
>> this is a temporary solution anyway and does not really hurt to leave a
>> few unnecessary ones around?
> 
> They will be removed. It doesn't hurt to keep them around, but they are 
> not necessary.


Lillian