Skip to content

Commit

Permalink
Remove warnings from the interface
Browse files Browse the repository at this point in the history
  • Loading branch information
SchaichAlonso committed Sep 11, 2023
1 parent ee14918 commit 338fbf6
Show file tree
Hide file tree
Showing 2 changed files with 74 additions and 0 deletions.
73 changes: 73 additions & 0 deletions ports/tinyorm/0004-remove-interface-warnings.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,73 @@
diff --git a/cmake/CommonModules/TinyCommon.cmake b/cmake/CommonModules/TinyCommon.cmake
index fd2cfc8a..160cecaa 100644
--- a/cmake/CommonModules/TinyCommon.cmake
+++ b/cmake/CommonModules/TinyCommon.cmake
@@ -91,7 +91,6 @@ ${TINY_UNPARSED_ARGUMENTS}")
/Zc:__cplusplus
# Standards-conforming behavior
/Zc:strictStrings
- /WX /W4
$<$<CONFIG:Debug>:/sdl>
)

@@ -102,7 +101,6 @@ ${TINY_UNPARSED_ARGUMENTS}")
/permissive-
# clang-cl 16 throws -Wunused-command-line-argument, so provide it
# only for the MSVC
- /guard:cf
/bigobj
# Standards-conforming behavior
/Zc:wchar_t,rvalueCast,inline
@@ -117,7 +115,6 @@ ${TINY_UNPARSED_ARGUMENTS}")
$<$<NOT:$<CONFIG:Debug>>:/OPT:REF,ICF=5>
# /OPT:REF,ICF does not support incremental linking
$<$<CONFIG:RelWithDebInfo>:/INCREMENTAL:NO>
- /WX
)
endif()

@@ -135,44 +132,6 @@ ${TINY_UNPARSED_ARGUMENTS}")
)
endif()

- if(NOT MSVC AND
- (CMAKE_CXX_COMPILER_ID STREQUAL "GNU" OR
- CMAKE_CXX_COMPILER_ID STREQUAL "Clang" OR
- CMAKE_CXX_COMPILER_ID STREQUAL "AppleClang")
- )
- target_compile_options(${target} INTERFACE
- # -fexceptions for linux is not needed, it is on by default
- -Wall
- -Wextra
- -Weffc++
- -Werror
- -Wfatal-errors
- -Wcast-qual
- -Wcast-align
- -Woverloaded-virtual
- -Wold-style-cast
- -Wshadow
- -Wundef
- -Wfloat-equal
- -Wformat-security
- -Wdouble-promotion
- -Wconversion
- -Wzero-as-null-pointer-constant
- -Wuninitialized
- -pedantic
- -pedantic-errors
- # Reduce I/O operations
- -pipe
- )
-
- # Clang 12 still doesn't support -Wstrict-null-sentinel
- include(CheckCXXCompilerFlag)
- check_cxx_compiler_flag(-Wstrict-null-sentinel SNS_SUPPORT)
- if(SNS_SUPPORT)
- target_compile_options(${target} INTERFACE -Wstrict-null-sentinel)
- endif()
- endif()
-
# Use faster lld linker on Clang (target the Clang except clang-cl with MSVC)
if(NOT MSVC AND CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
target_link_options(${target} INTERFACE -fuse-ld=lld)
1 change: 1 addition & 0 deletions ports/tinyorm/portfile.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ vcpkg_from_github(
0001-install-tom.patch
0002-comment.patch
0003-reduce-assert-scopes.patch
0004-remove-interface-warnings.patch
)

vcpkg_check_features(
Expand Down

0 comments on commit 338fbf6

Please sign in to comment.