The default bug view has changed. See this FAQ.

update graphite2 library to latest release

RESOLVED FIXED in mozilla14

Status

()

Core
Graphics: Text
RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: martin_hosken, Assigned: jfkthame)

Tracking

Trunk
mozilla14
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Due to a bug in the graphite2 engine, feature ids < 4 do not identify the right feature. This is fixed in the latest release of the graphite2 engine v1.1.2 which also has a few minor security bug fixes.

available from: http://sourceforge.net/projects/silgraphite/files/graphite2/graphite2-1.1.2.tgz/download or http://projects.palaso.org/attachments/download/227/graphite2-1.1.2.tgz
(Assignee)

Updated

5 years ago
Assignee: nobody → jfkthame
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Version: 14 Branch → Trunk
(Assignee)

Comment 1

5 years ago
Created attachment 616564 [details] [diff] [review]
patch, update graphite2 library to release 1.1.2
Attachment #616564 - Flags: review?(jdaggett)
(Assignee)

Comment 2

5 years ago
Tryserver job is at https://tbpl.mozilla.org/?tree=Try&rev=0a7ee097be10.

Comment 3

5 years ago
I think the title of the bug should be "update graphite to latest" or something like that.  Changes proposed for the 'font-feature-settings' syntax will mean that tags not four characters in length will be considered invalid.  See bug 718539, comment 34.

In standardizing how Graphite features are used on the web, I think we really should be encouraging developers to use existing OpenType tags where it makes sense.  For non-standard, or custom features, they can use the syntax for vendor-specific tags.  If it's really important to support a format specific to Graphite (or another *standardized* font technology) then adding syntax like 'graphite(feature-tag)' would make sense.  But right now I just don't see that as necessary, the existing pattern allows ample flexibility and we should encourage font developers to work with that pattern.

Comment 4

5 years ago
Comment on attachment 616564 [details] [diff] [review]
patch, update graphite2 library to release 1.1.2

Looks fine.

-#define GR2_VERSION_MAJOR   2
-#define GR2_VERSION_MINOR   0
-#define GR2_VERSION_BUGFIX  1
+#define GR2_VERSION_MAJOR   1
+#define GR2_VERSION_MINOR   1
+#define GR2_VERSION_BUGFIX  2

Is this change right?  The version is *decreasing*?!?
Attachment #616564 - Flags: review?(jdaggett) → review+
(Reporter)

Comment 5

5 years ago
Hmm. So later versions are worse than earlier ones. Ah well, just more evidence that the world is going to pot! In reality we just dropped the major version to sync with the public release number. Imagine an invisible epoch bump. It's a one off event. We are moving forward, honest :)

So, yes it is right.

Comment 6

5 years ago
Heh, ok.
(Assignee)

Updated

5 years ago
Summary: graphite fonts don't work with feature ids < 4 chars long → update graphite2 library to latest release
(Assignee)

Comment 7

5 years ago
Comment on attachment 616564 [details] [diff] [review]
patch, update graphite2 library to release 1.1.2

Requesting approval to land for mozilla-14.

This presents negligible risk to the mobile beta product, as (a) it's a low-risk library update anyhow; but also (b) the feature is preffed off by default, and therefore the update has no impact whatsoever except for those users who deliberately enable it in about:config.

It's desirable to take the update for the sake of the graphite user community who explicitly switch on the feature - the new release fixes a bug affecting access to certain font features, and includes several minor security/robustness fixes.

The upstream release was deliberately timed to fit in with our published schedule for mozilla 14, so as to get the font-feature fix into that release. Given the absence of risk to mobile, I think it should be accepted as normal.
Attachment #616564 - Flags: approval-mozilla-central?

Comment 8

5 years ago
Comment on attachment 616564 [details] [diff] [review]
patch, update graphite2 library to release 1.1.2

(In reply to Jonathan Kew (:jfkthame) from comment #7)
> It's desirable to take the update for the sake of the graphite user
> community who explicitly switch on the feature - the new release fixes a bug
> affecting access to certain font features, and includes several minor
> security/robustness fixes.

Approved for central since this is pref'd off.
Attachment #616564 - Flags: approval-mozilla-central? → approval-mozilla-central+
(Assignee)

Comment 9

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/e8f573bf8f2e
Target Milestone: --- → mozilla14
https://hg.mozilla.org/mozilla-central/rev/e8f573bf8f2e
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Blocks: 797863
Depends on: 721068
You need to log in before you can comment on or make changes to this bug.