Seems like this is not a problem with cmake builds?
I'm getting
-- The C compiler identification is Intel
-- The CXX compiler identification is Intel
-- Check for working C compiler: /export/home/opt/intel/bin/ia32/icc
-- Check for working C compiler: /export/home/opt/intel/bin/ia32/icc --
works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /export/home/opt/intel/bin/ia32/icpc
-- Check for working CXX compiler: /export/home/opt/intel/bin/ia32/icpc --
works
and all the -Wxxx flags are not enabled.
cmake version 2.6-patch 4
in environment:
CXX=icpc
CC=icc
$icc --version
icc (ICC) 11.0 20090131
Copyright (C) 1985-2009 Intel Corporation. All rights reserved.
$icpc --version
icpc (ICC) 11.0 20090131
Copyright (C) 1985-2009 Intel Corporation. All rights reserved.
On Tue, Dec 7, 2010 at 4:05 PM, Davi Arnaut <davi.arnaut@stripped> wrote:
> # At a local mysql-5.5-bugteam repository of davi
>
> 3171 Davi Arnaut 2010-12-07
> Bug#57991: Compiler flag change build error : adler32.c
>
> Do not use the same maintainer mode flags for both GCC and
> ICC. The -Wall option for ICC enables more warnings than its
> GCC counterpart. For now, only enable -Wall for ICC, but do
> not turn warnings into errors.
>
> added:
> cmake/maintainer.cmake
> modified:
> CMakeLists.txt
> === modified file 'CMakeLists.txt'
> --- a/CMakeLists.txt 2010-11-24 10:23:44 +0000
> +++ b/CMakeLists.txt 2010-12-07 15:05:28 +0000
> @@ -115,30 +115,22 @@ ENDIF()
> # Control aspects of the development environment which are
> # specific to MySQL maintainers and developers.
> #
> -INCLUDE (CheckCCompilerFlag)
> +INCLUDE(maintainer)
> OPTION(MYSQL_MAINTAINER_MODE "MySQL maintainer-specific development
> environment" OFF)
> -# Whether the maintainer mode should be enabled.
> +
> +# Whether the maintainer mode compiler options should be enabled.
> IF(MYSQL_MAINTAINER_MODE)
> - IF(CMAKE_COMPILER_IS_GNUCC)
> - CHECK_C_COMPILER_FLAG("-Wdeclaration-after-statement"
> HAVE_DECLARATION_AFTER_STATEMENT)
> - IF(HAVE_DECLARATION_AFTER_STATEMENT)
> - SET(MY_MAINTAINER_DECLARATION_AFTER_STATEMENT
> "-Wdeclaration-after-statement")
> - ENDIF()
> - SET(MY_MAINTAINER_C_WARNINGS
> - "-Wall -Wextra -Wunused -Wwrite-strings -Wno-strict-aliasing
> -Werror")
> + IF("${CMAKE_C_COMPILER_ID}" MATCHES "GNU")
> + SET_MYSQL_MAINTAINER_GNU_C_OPTIONS()
> ENDIF()
> - IF(CMAKE_COMPILER_IS_GNUCXX)
> - SET(MY_MAINTAINER_CXX_WARNINGS "${MY_MAINTAINER_C_WARNINGS}
> -Wno-unused-parameter"
> - CACHE STRING "C++ warning options used in maintainer builds.")
> + IF("${CMAKE_CXX_COMPILER_ID}" MATCHES "GNU")
> + SET_MYSQL_MAINTAINER_GNU_CXX_OPTIONS()
> ENDIF()
> - IF(CMAKE_COMPILER_IS_GNUCC)
> - SET(MY_MAINTAINER_C_WARNINGS
> - "${MY_MAINTAINER_C_WARNINGS}
> ${MY_MAINTAINER_DECLARATION_AFTER_STATEMENT}"
> - CACHE STRING "C warning options used in maintainer builds.")
> + IF("${CMAKE_C_COMPILER_ID}" MATCHES "Intel")
> + SET_MYSQL_MAINTAINER_INTEL_C_OPTIONS()
> ENDIF()
> - # Do not make warnings in checks into errors.
> - IF(CMAKE_COMPILER_IS_GNUCC AND CMAKE_COMPILER_IS_GNUCXX)
> - SET(CMAKE_REQUIRED_FLAGS "${CMAKE_REQUIRED_FLAGS} -Wno-error")
> + IF("${CMAKE_CXX_COMPILER_ID}" MATCHES "Intel")
> + SET_MYSQL_MAINTAINER_INTEL_CXX_OPTIONS()
> ENDIF()
> ENDIF()
>
>
> === added file 'cmake/maintainer.cmake'
> --- a/cmake/maintainer.cmake 1970-01-01 00:00:00 +0000
> +++ b/cmake/maintainer.cmake 2010-12-07 15:05:28 +0000
> @@ -0,0 +1,51 @@
> +# Copyright (c) 2010, Oracle and/or its affiliates. All rights reserved.
> +#
> +# This program is free software; you can redistribute it and/or modify
> +# it under the terms of the GNU General Public License as published by
> +# the Free Software Foundation; version 2 of the License.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301
> USA
> +
> +INCLUDE(CheckCCompilerFlag)
> +
> +# Setup GCC (GNU C compiler) warning options.
> +MACRO(SET_MYSQL_MAINTAINER_GNU_C_OPTIONS)
> + SET(MY_MAINTAINER_WARNINGS
> + "-Wall -Wextra -Wunused -Wwrite-strings -Wno-strict-aliasing
> -Werror")
> + CHECK_C_COMPILER_FLAG("-Wdeclaration-after-statement"
> HAVE_DECLARATION_AFTER_STATEMENT)
> + IF(HAVE_DECLARATION_AFTER_STATEMENT)
> + SET(MY_MAINTAINER_DECLARATION_AFTER_STATEMENT
> "-Wdeclaration-after-statement")
> + ENDIF()
> + SET(MY_MAINTAINER_C_WARNINGS
> + "${MY_MAINTAINER_WARNINGS}
> ${MY_MAINTAINER_DECLARATION_AFTER_STATEMENT}"
> + CACHE STRING "C warning options used in maintainer builds.")
> + # Do not make warnings in checks into errors.
> + SET(CMAKE_REQUIRED_FLAGS "${CMAKE_REQUIRED_FLAGS} -Wno-error")
> +ENDMACRO()
> +
> +# Setup G++ (GNU C++ compiler) warning options.
> +MACRO(SET_MYSQL_MAINTAINER_GNU_CXX_OPTIONS)
> + SET(MY_MAINTAINER_CXX_WARNINGS "${MY_MAINTAINER_WARNINGS}
> -Wno-unused-parameter"
> + CACHE STRING "C++ warning options used in maintainer builds.")
> +ENDMACRO()
> +
> +# Setup ICC (Intel C Compiler) warning options.
> +MACRO(SET_MYSQL_MAINTAINER_INTEL_C_OPTIONS)
> + SET(MY_MAINTAINER_WARNINGS "-Wall")
> + SET(MY_MAINTAINER_C_WARNINGS "${MY_MAINTAINER_WARNINGS}"
> + CACHE STRING "C warning options used in maintainer builds.")
> +ENDMACRO()
> +
> +# Setup ICPC (Intel C++ Compiler) warning options.
> +MACRO(SET_MYSQL_MAINTAINER_INTEL_CXX_OPTIONS)
> + SET(MY_MAINTAINER_CXX_WARNINGS "${MY_MAINTAINER_WARNINGS}"
> + CACHE STRING "C++ warning options used in maintainer builds.")
> +ENDMACRO()
> +
>
>
>
> --
> MySQL Code Commits Mailing List
> For list archives: http://lists.mysql.com/commits
> To unsubscribe:
> http://lists.mysql.com/commits?unsub=1
>
