Closed Bug 750747 Opened 9 years ago Closed 9 years ago

Rip out libpng from libxul

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla15

People

(Reporter: ehsan, Assigned: ehsan)

References

Details

Attachments

(1 file)

libpng needs to go outside of libxul, too!
Try run for 1ddf3f87d0bc is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=1ddf3f87d0bc
Results (out of 104 total builds):
    exception: 23
    success: 53
    warnings: 11
    failure: 17
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/eakhgari@mozilla.com-1ddf3f87d0bc
Attached patch Patch (v1)Splinter Review
This patch moves libpng into gkmedias.dll.
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #620043 - Flags: review?(ted.mielczarek)
Attachment #620043 - Flags: review?(mh+mozilla)
Comment on attachment 620043 [details] [diff] [review]
Patch (v1)

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

::: layout/media/symbols.def.in
@@ +106,5 @@
>  ShGetInfo
>  ShConstructCompiler
>  ShGetActiveAttrib
>  ShGetActiveUniform
> +#ifndef MOZ_NATIVE_PNG

Honestly I'm really not interested in supporting native PNG on windows, but I guess it doesn't hurt to leave it here.
Attachment #620043 - Flags: review?(ted.mielczarek)
Attachment #620043 - Flags: review?(mh+mozilla)
Attachment #620043 - Flags: review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/e58855b90cf6
Target Milestone: --- → mozilla15
This broke the build, so I backed it out for now: https://hg.mozilla.org/integration/mozilla-inbound/rev/75de3dfde0bd
Seems like that was some sort of a build system hick-up, relanded:

https://hg.mozilla.org/integration/mozilla-inbound/rev/b60dc9ae8aae
This one surprises me.  I thought gkmedias.dll was for things unlikely to be used during startup.
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.