Le 21/02/2012 21:51, Andrew Benton a écrit : > On Tue, 21 Feb 2012 09:47:00 +0100 > Pierre Labastie<pierre.labas...@neuf.fr> wrote: > >> Hi, >> >> I have done a test of LFS-7.1-rc1. ICA went OK, except the >> already reported problem with ld.so.cache (ldconfig still missing >> somewhere), which is not a big issue. In case somebody else >> does ICA, there is this difference in etip.h between ICA >> iterations 1 and 2: >> ----------------------------------- >> --- iteration-1/usr/include/etip.h 2012-02-20 10:38:14.000000000 +0100 >> +++ iteration-2/usr/include/etip.h 2012-02-20 11:35:57.000000000 +0100 >> @@ -62,7 +62,7 @@ >> #endif >> >> #ifndef ETIP_NEEDS_MATH_H >> -#define ETIP_NEEDS_MATH_H 0 >> +#define ETIP_NEEDS_MATH_H 1 >> #endif >> >> #ifndef ETIP_NEEDS_MATH_EXCEPTION >> ----------------------------------- >> It seems to be due to a bug in ncurses' configure: see the >> thread starting at: >> http://lists.gnu.org/archive/html/bug-ncurses/2012-02/msg00003.html > My last message was wrong. I get the same results on i686 and x86_64. > If ncurses is already installed and I don't apply your patch I get: > #define ETIP_NEEDS_MATH_H 1 > If ncurses is already installed and I apply your patch I get: > #define ETIP_NEEDS_MATH_H 0 > If ncurses is not installed you patch makes no difference, I still get: > define ETIP_NEEDS_MATH_H 0 > > Could it be that ETIP_NEEDS_MATH_H 0 (the current situation) is the > correct result and the result you get when reinstalling ncurses is the > wrong one? > > Andy I think so. But in the third situation, it arrives to the good result for bad reasons: The test program is a small file which basically includes etip.h.in, which itself calls ncurses_dll_h, which is a generated file, hence does not exist at the time configure is run. (if ncurses is already installed, the host's ncurses_dll.h is found instead). So all the tests fail and in this case, it ends up with #define ETIP_NEED_MATH_H 0
Should be corrected in current upstream patch list. Do not know if you LFS devs think it is a big issue (having math.h needlessly included when ncurses C++ bindings are used). I suggest waiting for the next release of ncurses. Regards Pierre -- http://linuxfromscratch.org/mailman/listinfo/lfs-dev FAQ: http://www.linuxfromscratch.org/faq/ Unsubscribe: See the above information page
