| Message ID | 1399626447-2540-1-git-send-email-s.martin49@gmail.com |
|---|---|
| State | Accepted |
| Commit | f3b1b92d321b4b21c35e6f8ac27f076edd268639 |
| Headers | show |
>>>>> "Samuel" == Samuel Martin <s.martin49@gmail.com> writes: > During the CMake bootstrap phase, the {C,LD}FLAGS set in the bootstrap > environment are not forwarded/converted as CMake flags. > The CMake build contains a bootstrap phase building a minimal CMake > program using a standard Makfile, then reconfigures itself with this > minimal program. > On system with no ncurses installed, and because the prefix option > points to $(HOST_DIR)/usr, if host-cmake was built after host-ncurses, > then ncurses libraries and headers are correctly found (in the host > tree) during the second configuration (because of the prefix). However, > it fails at building ccmake (the curses interface) because the > CMAKE_C_FLAGS, CMAKE_CXX_FLAGS and CMAKE_EXE_LINKER_FLAGS do not > point to the host tree. > Because these flags are needed when running the bootstrap script, > this patch makes sure the same flags are set when running the second > configuration. > Reported-by: Fabio Porcedda <fabio.porcedda@gmail.com> > Signed-off-by: Samuel Martin <s.martin49@gmail.com> > Cc: Fabio Porcedda <fabio.porcedda@gmail.com> > Cc: Luca Ceresoli <luca@lucaceresoli.net> > --- > changes v1 -> v2: > - use HOST_CXXFLAGS (Fabio) > - cut lines at <=80 char. (Fabio) > - fix typo in commit log (Luca) Committed, thanks.
diff --git a/package/cmake/cmake.mk b/package/cmake/cmake.mk index d45c642..235988b 100644 --- a/package/cmake/cmake.mk +++ b/package/cmake/cmake.mk @@ -14,7 +14,11 @@ define HOST_CMAKE_CONFIGURE_CMDS (cd $(@D); \ LDFLAGS="$(HOST_LDFLAGS)" \ CFLAGS="$(HOST_CFLAGS)" \ - ./bootstrap --prefix=$(HOST_DIR)/usr --parallel=$(PARALLEL_JOBS) \ + ./bootstrap --prefix=$(HOST_DIR)/usr \ + --parallel=$(PARALLEL_JOBS) -- \ + -DCMAKE_C_FLAGS="$(HOST_CFLAGS)" \ + -DCMAKE_CXX_FLAGS="$(HOST_CXXFLAGS)" \ + -DCMAKE_EXE_LINKER_FLAGS="$(HOST_LDFLAGS)" \ ) endef
During the CMake bootstrap phase, the {C,LD}FLAGS set in the bootstrap environment are not forwarded/converted as CMake flags. The CMake build contains a bootstrap phase building a minimal CMake program using a standard Makfile, then reconfigures itself with this minimal program. On system with no ncurses installed, and because the prefix option points to $(HOST_DIR)/usr, if host-cmake was built after host-ncurses, then ncurses libraries and headers are correctly found (in the host tree) during the second configuration (because of the prefix). However, it fails at building ccmake (the curses interface) because the CMAKE_C_FLAGS, CMAKE_CXX_FLAGS and CMAKE_EXE_LINKER_FLAGS do not point to the host tree. Because these flags are needed when running the bootstrap script, this patch makes sure the same flags are set when running the second configuration. Reported-by: Fabio Porcedda <fabio.porcedda@gmail.com> Signed-off-by: Samuel Martin <s.martin49@gmail.com> Cc: Fabio Porcedda <fabio.porcedda@gmail.com> Cc: Luca Ceresoli <luca@lucaceresoli.net> --- changes v1 -> v2: - use HOST_CXXFLAGS (Fabio) - cut lines at <=80 char. (Fabio) - fix typo in commit log (Luca) --- package/cmake/cmake.mk | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)