Closed Bug 1342964 Opened 7 years ago Closed 7 years ago

Update Harfbuzz to 1.4.3

Categories

(Core :: Graphics: Text, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: tjr, Assigned: RyanVM)

References

Details

(Whiteboard: [third-party-lib-audit])

Attachments

(1 file, 1 obsolete file)

This is a (semi-)automated bug making you aware that there is an available upgrade for an embedded third-party library.

Harfbuzz is currently at version 1.4.2 in mozilla-central, and the latest version of the library released is 1.4.3. 

I fetched the latest version of the library from https://github.com/behdad/harfbuzz/.

You can leave this bug open, and it will be updated if a newer version of the library becomes available. If you close it as WONTFIX, please indicate if you do not wish to receive any future bugs upon new releases of the library.
Assignee: nobody → ryanvm
First attempt at a Try push isn't going well:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=4726ed40db1fbcec2938a25ebcea09b012c6d40a

https://treeherder.mozilla.org/logviewer.html#?job_id=80490034&repo=try

gfx/harfbuzz/src/hb-fallback-shape.cc:31:237: error: 'struct hb_shaper_data_t' has no member named 'fallback'
gfx/harfbuzz/src/hb-fallback-shape.cc:31:348: error: '_hb_fallback_shaper_face_data_create' was not declared in this scope
gfx/harfbuzz/src/hb-fallback-shape.cc:31:550: error: 'struct hb_shaper_data_t' has no member named 'fallback'
gfx/harfbuzz/src/hb-fallback-shape.cc:31:670: error: 'struct hb_shaper_data_t' has no member named 'fallback'
gfx/harfbuzz/src/hb-fallback-shape.cc:31:824: error: '_hb_fallback_shaper_face_data_destroy' was not declared in this scope
gfx/harfbuzz/src/hb-fallback-shape.cc:32:237: error: 'struct hb_shaper_data_t' has no member named 'fallback'
gfx/harfbuzz/src/hb-fallback-shape.cc:32:348: error: '_hb_fallback_shaper_font_data_create' was not declared in this scope
gfx/harfbuzz/src/hb-fallback-shape.cc:32:550: error: 'struct hb_shaper_data_t' has no member named 'fallback'
gfx/harfbuzz/src/hb-fallback-shape.cc:32:670: error: 'struct hb_shaper_data_t' has no member named 'fallback'
gfx/harfbuzz/src/hb-fallback-shape.cc:32:824: error: '_hb_fallback_shaper_font_data_destroy' was not declared in this scope
That's because you have not defined HAVE_FALLBACK but are compiling hb-fallback-shape.cc.  Remove that file from compile list; it's unused by Firefox.  Or define HAVE_FALLBACK instead. I marginally prefer the latter.
(In reply to Behdad Esfahbod from comment #2)
> That's because you have not defined HAVE_FALLBACK but are compiling
> hb-fallback-shape.cc.  Remove that file from compile list; it's unused by
> Firefox.  Or define HAVE_FALLBACK instead. I marginally prefer the latter.

Curious why you prefer that -- are you aware of cases where we might need it?
(In reply to Jonathan Kew (:jfkthame) from comment #4)
> (In reply to Behdad Esfahbod from comment #2)
> > That's because you have not defined HAVE_FALLBACK but are compiling
> > hb-fallback-shape.cc.  Remove that file from compile list; it's unused by
> > Firefox.  Or define HAVE_FALLBACK instead. I marginally prefer the latter.
> 
> Curious why you prefer that -- are you aware of cases where we might need it?

In the past, in at least one occasion I set HB_SHAPER_LIST env var and checked Firefox behavior in a useful way. That doesn't work anymore, because HAVE_FALLBACK is undefined now.  It will be useful as baseline for performance measurements, but I don't see myself using it anytime soon. So, yeah, feel free to just not compile.
Removes hb-fallback-shape.cc from moz.build per the earlier discussion. Looking green on Try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=955145cfa9a2fa7edd548be9d45b47090df51683&group_state=expanded
Attachment #8841855 - Attachment is obsolete: true
Attachment #8841855 - Flags: review?(jfkthame)
Attachment #8842447 - Flags: review?(jfkthame)
Comment on attachment 8842447 [details] [diff] [review]
update harfbuzz to version 1.4.3

Review of attachment 8842447 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM
Attachment #8842447 - Flags: review?(jfkthame) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/39a67f6289c4
Update harfbuzz to version 1.4.3. r=jfkthame
https://hg.mozilla.org/mozilla-central/rev/39a67f6289c4
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Blocks: 1344578
Whiteboard: [third-party-lib-audit]
You need to log in before you can comment on or make changes to this bug.