1049249 - Remove app_libxul_dirs

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Mike Hommey [:glandium]]

It was used to force building components after libxul. That's not necessary anymore with USE_LIBS \o/

Comment 1

[Mike Hommey [:glandium]]

(In reply to Mike Hommey [:glandium] from comment #0)
> It was used to force building components after libxul.

s/after/before/

> That's not necessary anymore with USE_LIBS \o/

s/USE_LIBS/FINAL_LIBRARY/

But that doesn't change that it's not required anymore.

Comment 2

[Mike Hommey [:glandium]]

Attachment: Remove app_libxul_dirs

Comment 3

[Mike Hommey [:glandium]]

Attachment: Remove app_libxul_dirs from c-c

Comment 4

[Mike Hommey [:glandium]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5bdf577bf6e0

Comment 5

[Nigel Babu [:nigelb]]

https://hg.mozilla.org/mozilla-central/rev/5bdf577bf6e0

Comment 6

[Joshua Cranmer [:jcranmer]]

Comment on attachment 8468149 [details] [diff] [review]
Remove app_libxul_dirs from c-c

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

Such bitrotting...

Comment 7

[Mike Hommey [:glandium]]

https://hg.mozilla.org/comm-central/rev/1c0a95ffa495