943197 - Remove useless -DIMPL_LIBXUL

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Comment 1

[Mike Hommey [:glandium]]

Attachment: Remove useless -DIMPL_LIBXUL

Comment 2

[Nathan Froyd [:froydnj]]

I saw this the other week and also thought about removing it, but I couldn't convince myself that it didn't somehow affect files compiled in those directories (various things in xpcom/ care about IMPL_LIBXUL).  Did you verify that it doesn't, or did I just not look far enough down the rabbit hole to understand how everything fits together?

Comment 3

[Michael Shal [:mshal]]

I tried changing these defines to -DMARF_IMPL_LIBXUL, and then added #warning lines in the xpcom headers where IMPL_LIBXUL is used if MARF_IMPL_LIBXUL was defined. It looks like nscore.h is included by xpcom/string/src/nsUTF8UtilsSSE2.cpp and xpcom/reflect/xptcall/src/md/unix/xptcinvoke_x86_64_unix.cpp, so I think those two directories at least require some further investigation.

Comment 4

[Mike Hommey [:glandium]]

Apart from xpcom/typelib/xpt/tests/Makefile.in, which i think should actually not be changed, all the others are for things that end up in libxul by force of FINAL_LIBRARY, possibly transitively. config/config.mk defines LIBXUL_LIBRARY at makefile level when FINAL_LIBRARY in backend.mk is xul (which it is for everything that has a FINAL_LIBRARY in moz.build that ends up being xul if it is the FINAL_LIBRARY of the FINAL_LIBRARY (...)). And LIBXUL_LIBRARY defined at makefile level makes IMPL_LIBXUL defined at C/C++ level.

Comment 5

[Michael Shal [:mshal]]

(In reply to Mike Hommey [:glandium] from comment #4)
> Apart from xpcom/typelib/xpt/tests/Makefile.in, which i think should
> actually not be changed, all the others are for things that end up in libxul
> by force of FINAL_LIBRARY, possibly transitively. config/config.mk defines
> LIBXUL_LIBRARY at makefile level when FINAL_LIBRARY in backend.mk is xul
> (which it is for everything that has a FINAL_LIBRARY in moz.build that ends
> up being xul if it is the FINAL_LIBRARY of the FINAL_LIBRARY (...)). And
> LIBXUL_LIBRARY defined at makefile level makes IMPL_LIBXUL defined at C/C++
> level.

Works for me - thanks for the clarification!

Comment 6

[Mike Hommey [:glandium]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e044bc75f8a8

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/e044bc75f8a8