[OpenJDK 2D-Dev] [PATCH FOR REVIEW] fix for bug 8011693: Remove redundant fontconfig files

[Phil Race]

Jiri,

I think this has mostly been hashed out as the fix is reduced to Linux
but here's my over-due input :

1) Windows *absolutely* still needs fontconfig files.

2) Mac OS X doesn't obey what's there but that doesn't mean its
going to work when you just remove them.

3) Solaris *does* want the Solaris one, even though it has the
same fontconfig platform support as Linux. This is for compatibility.

4) Linux does not need them *so long as* fontconfig is there and working.
OpenJDK on any Linux desktop should be fine.

5) However policy decisions were made to leave them there for some
particular linux variants in the closed repo, again for compatibility,
but you are leaving that alone, so that's fine, although we should 
revisit this

6) I never noticed the bsd one before. It must have snuck in with mac port.
Is anything even referencing it ? If not I think this can be removed too.

7) The files themselves and support for the files are distinct issues.
There should still be the ability for [say] Gentoo, to decide they want
a particular set of fonts used and so they will ship a file .
So it might be better to leave the variables there (empty) and with
a comment that this is a placeholder.

-phil.

On 5/29/2013 5:06 AM, Andrew Hughes wrote:
> ----- Original Message -----
>> On 05/20/2013 04:37 PM, Jiri Vanek wrote:
>>> On 05/10/2013 04:08 PM, Jiri Vanek wrote:
>>>> On 04/08/2013 05:31 PM, Jiri Vanek wrote:
>>>>> On 04/08/2013 04:13 PM, Vladislav Karnaukhov wrote:
>>>>>> Hello Jiri,
>>>>>>
>>>>>> please see inline.
>>>>>>
>>>>>> On 4/8/2013 05:29 PM, Jiri Vanek wrote:
>>>>>>> On 04/08/2013 02:39 PM, Vladislav Karnaukhov wrote:
>>>>>>>
>>>>>>> Thank you very much for win-check! It will force me to install new
>>>>>>> windows machine somewhere.
>>>>>>> Do you mind do check if pure removal of fontconfig files (both src and
>>>>>>> bfc)  from you installed jdk7/8 on windows will work? (should)
>>>>>> Yes, I've checked and it does *not* work. That's the reason why I
>>>>>> replied to your very first
>>>>>> message. A removal of fontconfig.* files simply crashes Java, - on both
>>>>>> Windows and Mac, - because
>>>>>> some font management-related classes rely on these files. Hence my
>>>>>> question regarding deeper
>>>>>> re-design on font management system...
>>>>>>
>>>>>> I've tested Mac build as well, and there's the same error:
>>>>> Ok. I will try anyway:)
>>>>> For linux I'm quite sure the new fontmanagers are working pretty fine.
>>>>> Do you think it will be acceptable to prepare smaller clean up - to
>>>>> remove all linux fontconfig
>>>>> files?
>>>>>
>>>>> And later, as separate changeset to  fontmanagers for windows/mac, but
>>>>> I'm afraid I will not be
>>>>> capable of such an development on non linux system.
>>>>>
>>>>> Thanx for your help,
>>>>>
>>>>>    J.
>>>> Hi!
>>>>
>>>> I had finally found some free time, so here it is - smaller version which
>>>> is removing just stuff for
>>>> linux when OpenJDK is defined.
>>>>
>>>> http://jvanek.fedorapeople.org/oracle/jdk8/webrevs/removedFontConfigFiles-linuxOnly/
>>>>
>>>> Although I had windows build, I lost this machine so - again (and sorry
>>>> for that) - tested only on
>>>> Fedora.
>>>>
>>>> Also when I read the individual fontmanagers, I believe that they really
>>>> *should* work without
>>>> fontocfigs. So although this is fixing the 8011693, new bugs should be
>>>> filled for windows and mac,
>>>> because theirs implementations are broken.
>>>>
>>>> Thank you very much for any comments.
>>>>
>>>>      Best Regards
>>>>          j.
>>>
>>    Ping?
>>
>> Any advice how to move this forward?
>>
>>> I know that this is minor fix compared to others I can read on this
>>> channel, but as the font
>>> managers exists, and fontconfig files *should* be redundant, then this
>>> change should be done. If
>>> fontmanagers are buggy (and eg windows one appeared to be) then as soon as
>>> this will be tempted then
>>> sooner it will get fixed.
>>>
>>> For linux I'm pretty sure this is working, and we have even removed the
>>> fontconfig files from
>>> packages in public facing version three months ago [1]
>>>
>>> So this can be first step to get rid of old and redundant font mapping
>>> completely.
>>>
>>>
>>> J.
>>>
>>>
>>> [1]
>>> http://pkgs.fedoraproject.org/cgit/java-1.7.0-openjdk.git/commit/?h=f17&id=9d6dd62ae2123635b4d15e40e527a0b617756484
>>>
>>> (search for +rm %{buildoutputdir}/j2re-image/lib/fontconfig)
>>
> I've applied this patch and built OpenJDK, and it went fine.  A basic Swing
> application still loads up fine.
>
> So looks good to go to me.