947379 - Remove MOZ_PANGO

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Jeff Muizelaar [:jrmuizel]]

Attachment: Remove MOZ_PANGO

We I don't believe we've been able to build without MOZ_PANGO and with MOZ_WIDGET_GTK since 2008. It feels like it's time for MOZ_PANGO to go.

Comment 1

[Jonathan Kew [:jfkthame]]

Aww... when I saw this bug's title, my first thought was that you were going to rip out Pango support altogether.

Yes, AFAIK building on Linux/Gtk with --disable-pango has been broken for quite a while. We've looked at this a bit, e.g. in bug 703042 and bug 754506, but never actually spent the time to finish the job. (Sorry!)

Before we decide to do this, I'd like to check with Oleg: do you still have any interest or use-case for the --disable-pango option, or can we assume that it's no longer important, as you haven't been nagging me about it? :)

Comment 2

[Oleg Romashin (:romaxa)]

Build without pango is not important for me anymore, the only case where it might be needed is LinuxGL port (where I would like to use as minimum dependencies as possible), but with that port we can use Android font backend pretty much as is.

So I'm fine with this define removal.

Comment 3

[Jonathan Kew [:jfkthame]]

Comment on attachment 8343913 [details] [diff] [review]
Remove MOZ_PANGO

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

OK, let's do it. Less code, more better. :)

Bonus points for a followup bug to entirely remove our dependency on pango, and rename gfxPangoFonts to gfxFcFonts or something like that. I think we're basically only using it for some pango_language APIs these days; would be nice if we could drop it altogether.

Comment 4

[Jonathan Kew [:jfkthame]]

Hmm, just a sec - shouldn't you also be removing the --disable-pango option from configure?

Comment 5

[Jeff Muizelaar [:jrmuizel]]

(In reply to Jonathan Kew (:jfkthame) from comment #3)
> Comment on attachment 8343913 [details] [diff] [review]
> Remove MOZ_PANGO
> 
> Review of attachment 8343913 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> OK, let's do it. Less code, more better. :)
> 
> Bonus points for a followup bug to entirely remove our dependency on pango,
> and rename gfxPangoFonts to gfxFcFonts or something like that. I think we're
> basically only using it for some pango_language APIs these days; would be
> nice if we could drop it altogether.

The only interesting part that we're using is:
pango_language_includes_script() followed by pango_script_get_sample_language(). This gives us a language string that we can pass to Fontconfig to do the lookups. Is there something different/better we can do to replace this.

Comment 6

[Jeff Muizelaar [:jrmuizel]]

Attachment: Remove MOZ_PANGO

This is with the configure changes.

Comment 7

[Jeff Muizelaar [:jrmuizel]]

https://tbpl.mozilla.org/?tree=Try&rev=b123d9d9b37d

Comment 8

[Karl Tomlinson (:karlt)]

(In reply to Jonathan Kew (:jfkthame) from comment #3)
> and rename gfxPangoFonts to gfxFcFonts or something like that.

That would make sense.

> I think we're
> basically only using it for some pango_language APIs these days; would be
> nice if we could drop it altogether.

We're going to load libpango* anyway because GTK depends on them, and we will need Pango to get system font info from GTK.

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/4ab015ae5eaf