Closed Bug 750728 Opened 8 years ago Closed 8 years ago

Rip out graphite from libxul

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla15

People

(Reporter: ehsan, Assigned: jfkthame)

References

Details

Attachments

(2 files)

It's that time of the year again.  We need to rip out stuff from libxul.  See bug 750661.
Attachment #619931 - Flags: review?(ted.mielczarek)
Attachment #619931 - Flags: review?(ted.mielczarek) → review+
Something like this ought to work, I think - have pushed it to tryserver, and will request review if it builds ok.
(In reply to Jonathan Kew (:jfkthame) from comment #2)
> Created attachment 619945 [details] [diff] [review]
> patch, restore graphite but move it to gkmedias.dll on windows
> 
> Something like this ought to work, I think - have pushed it to tryserver,
> and will request review if it builds ok.

You may have to list exported symbols in layout/media/symbols.def.in
I don't think that's needed, because of the use of GRAPHITE2_EXPORTING to get the appropriate dllexport attributes added.

Tryserver builds at https://tbpl.mozilla.org/?tree=Try&rev=24f57e880886 look happy on all desktop platforms; still waiting for android to catch up, but I believe this'll be ok.
Attachment #619945 - Flags: review?(ted.mielczarek)
https://hg.mozilla.org/mozilla-central/rev/bfa638e5df16
https://hg.mozilla.org/mozilla-central/rev/83ff77ce8d6c
Assignee: nobody → jfkthame
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.