Skip to content

Commit

Permalink
Build with -Wall -Werror now. Three directories still have unresolved…
Browse files Browse the repository at this point in the history
… warnings so this behaviour is disabled there (providers/wfs prviders/grass and plugins/grass). A nasty message is displayed during cmake configure process to remind that these errors must be resolved.

git-svn-id: http://svn.osgeo.org/qgis/trunk@6626 c8812cc2-4d05-0410-92ff-de0c093fc19c
  • Loading branch information
timlinux committed Feb 18, 2007
1 parent 5d6c20b commit 45b012d
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 3 deletions.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Expand Up @@ -114,7 +114,7 @@ FIND_PROGRAM(QT_LRELEASE_EXECUTABLE
#############################################################
# enable warnings

ADD_DEFINITIONS( -Wall )
ADD_DEFINITIONS( -Wall -Werror )

IF (CMAKE_BUILD_TYPE MATCHES Debug)
ADD_DEFINITIONS(-DQGISDEBUG=1)
Expand Down
2 changes: 1 addition & 1 deletion src/core/CMakeLists.txt
Expand Up @@ -69,7 +69,7 @@ ADD_FLEX_FILES(QGIS_CORE_SRCS qgssearchstringlexer.ll)

ADD_BISON_FILES(QGIS_CORE_SRCS qgssearchstringparser.yy)

# Disabe compiler warnings for lex generated sources
# Disable compiler warnings for lex generated sources
SET_SOURCE_FILES_PROPERTIES(
${CMAKE_BINARY_DIR}/src/core/flex_qgssearchstringlexer.cpp
PROPERTIES COMPILE_FLAGS -w)
Expand Down
2 changes: 2 additions & 0 deletions src/plugins/grass/CMakeLists.txt
Expand Up @@ -3,6 +3,8 @@ SUBDIRS(config modules themes)

ADD_DEFINITIONS(-DGRASS_BASE=\\\"${GRASS_PREFIX}\\\")
ADD_DEFINITIONS(-DHAVE_OPENPTY=${HAVE_OPENPTY})
MESSAGE("plugins/grass : -Werror removed for this directory for now - please get rid of all warnings!")
REMOVE_DEFINITIONS( -Werror )

########################################################
# Files
Expand Down
2 changes: 2 additions & 0 deletions src/providers/grass/CMakeLists.txt
@@ -1,5 +1,7 @@

ADD_DEFINITIONS(-DGRASS_BASE=\\\"${GRASS_PREFIX}\\\")
MESSAGE("providers/grass : -Werror removed for this directory for now - please get rid of all warnings!")
REMOVE_DEFINITIONS( -Werror )

########################################################
# Files
Expand Down
2 changes: 1 addition & 1 deletion src/providers/wfs/CMakeLists.txt
Expand Up @@ -8,7 +8,7 @@ SET (WFS_MOC_HDRS
qgswfsdata.h
)

MESSAGE("-Werror removed for this directory for now - please get rid of all warnings!")
MESSAGE("providers/wfs : -Werror removed for this directory for now - please get rid of all warnings!")
REMOVE_DEFINITIONS( -Werror )

########################################################
Expand Down
4 changes: 4 additions & 0 deletions tools/mapserver_export/CMakeLists.txt
Expand Up @@ -12,6 +12,10 @@ SET (MSEXPORT_MOC_HDRS qgsmapserverexport.h)

SET (MSEXPORT_UIS qgsmapserverexportbase.ui)

# Disable compiler warnings for lex generated sources
SET_SOURCE_FILES_PROPERTIES(
${CMAKE_CURRENT_SOURCE_DIR}/msexport_wrap.cxx
PROPERTIES COMPILE_FLAGS -w)

########################################################
# Build
Expand Down

0 comments on commit 45b012d

Please sign in to comment.