Closed
Bug 1201712
Opened 9 years ago
Closed 9 years ago
Package Firefox OS fonts
Categories
(B2GDroid Graveyard :: General, defect)
Tracking
(firefox43 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: fabrice, Assigned: fabrice)
Details
Attachments
(1 file)
(deleted),
patch
|
nalexander
:
review+
|
Details | Diff | Splinter Review |
We need to package the moztt fonts.
Assignee | ||
Comment 1•9 years ago
|
||
This add fonts provided by moztt.
Assignee: nobody → fabrice
Attachment #8656839 -
Flags: review?(nalexander)
Comment 2•9 years ago
|
||
Comment on attachment 8656839 [details] [diff] [review]
moztt-fonts.patch
Review of attachment 8656839 [details] [diff] [review]:
-----------------------------------------------------------------
I'm fine with this, but consider modifying https://dxr.mozilla.org/mozilla-central/source/mobile/android/b2gdroid/installer/package-manifest.in?offset=200#569 instead. I don't understand why this doesn't Just Work the same as b2g, given that https://dxr.mozilla.org/mozilla-central/source/b2g/installer/package-manifest.in?offset=200#834 exists. Aha! It's probably
#ifdef PACKAGE_MOZTT
@RESPATH@/fonts/*
#endif
later in that file. Consider following suit.
Attachment #8656839 -
Flags: review?(nalexander) → review+
Comment 4•9 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•