Bug 122568 – [Win] Check for MathML Fonts and warn user if they are missing

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Brent Fulgham]

Users who wish to use the MathML feature on Windows need to make sure they have the proper fonts installed.

Several math fonts are available in a convenient installer from the Mozilla site (https://developer.mozilla.org/@api/deki/files/6388/=MathML-fonts.msi).

We should check for our key MathML fonts and notify the user if they are missing so they can resolve this problem before attempting to use the feature.

[Brent Fulgham]

Created attachment 213805 [details]
Patch

[Frédéric Wang (:fredw)]

I'm wondering if the link to the Windows installer is really stable. In particular, it might change if Mozilla folks add/update fonts. Perhaps a better URL is https://developer.mozilla.org/en-US/docs/Mozilla/MathML_Project/Fonts

[Brent Fulgham]

(In reply to comment #2)
> I'm wondering if the link to the Windows installer is really stable. In particular, it might change if Mozilla folks add/update fonts. Perhaps a better URL is https://developer.mozilla.org/en-US/docs/Mozilla/MathML_Project/Fonts

That seems like a good idea.  I found that the site will actually redirect you properly (based on language settings) to the appropriate page, so I propose:

https://developer.mozilla.org/Mozilla/MathML_Project/Fonts

[Brent Fulgham]

Created attachment 213822 [details]
Patch

[Tim Horton]

Comment on attachment 213822 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=213822&action=review

> Tools/Scripts/webkitdirs.pm:1494
>      chomp($svnVersion);
> -    if (!$? and $svnVersion =~ /1\.8\./) {
> +    if (!$? and $svnVersion =~ /1\.8e./) {
>          print "svn 1.7.10 is known to be compatible with our servers. You are running $svnVersion,\nwhich may not work properly.\n"

No.

[Brent Fulgham]

Created attachment 213824 [details]
Patch

[Tim Horton]

Comment on attachment 213824 [details]
Patch

Better!

[Brent Fulgham]

Committed r157191: <http://trac.webkit.org/changeset/157191>