Le 21 sept. 2010 à 15:20, Quentin Mathé a écrit :

On Mon, Sep 20, 2010 at 12:44 PM, Fred Kiefer <fredkie...@gmx.de> wrote:

Am 19.09.2010 21:53, schrieb Quentin Mathé:

> To keep the compilation time as it is, we can pass
> -DCF_EXCLUDE_CSTD_HEADERS in Base, Gui and Back and continue to include > the C headers as we currently do it. We just need to include string.h in
> few more places if we do that.

Sounds like a great idea to me. Up to now we made sure that GNUstep only
includes the headers we really need and we should keep it that way for
the internal code at least.

I attached a patch that restores the old compilation time for Base. Before committing it, I just wanted to know whether the way I did it was ok…

This patch is incorrect.
I didn't realize that base.make is installed in Library/Makefiles/ Additions and imported by various gnustep-make targets such as framework. So this results in having -DCF_EXCLUDE_CSTD_HEADERS automatically passed to every project that links gnustep-base.

The right fix would be to put ADDITIONAL_CPPFLAGS += - DCF_EXCLUDE_CSTD_HEADERS in config.mak.in rather than base.mak.in. Alternatively we can explicitly define DCF_EXCLUDE_CSTD_HEADERS in every subprojects of Base, Gui and Back but this looks less clean and more tedious.

Quentin.
_______________________________________________
Gnustep-dev mailing list
Gnustep-dev@gnu.org
http://lists.gnu.org/mailman/listinfo/gnustep-dev

Reply via email to