Skip to content

Commit

Permalink
debian/patch: Relax compiler to continue on warnings (Closes: #957364)
Browse files Browse the repository at this point in the history
Disable error on warning in cmake files

Upstream FTBFS this in jerryscript submodule
but iotjs is not supporting it yet.

Relate-to: jerryscript-project#1945
Signed-off-by: Philippe Coval <[email protected]>
Change-Id: I4d84de2e826cc259d0f1cff75c797ec70cb1db65

Revert "debian/rules: Relax compiler to continue on warnings (Closes: #957364)"

This reverts commit 9264ca24a936edef4d1c02933a3fd97e6a469db2.
  • Loading branch information
rzr committed Aug 6, 2020
1 parent a3ee6e5 commit c738b38
Show file tree
Hide file tree
Showing 2 changed files with 32 additions and 0 deletions.
31 changes: 31 additions & 0 deletions debian/patches/0004-build-Disable-error-on-warning.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
From: Philippe Coval <[email protected]>
Date: Tue, 28 Jul 2020 16:48:29 +0200
Subject: build: Disable error on warning

Signed-off-by: Philippe Coval <[email protected]>
---
deps/jerry/CMakeLists.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/deps/jerry/CMakeLists.txt b/deps/jerry/CMakeLists.txt
index d27997c..eeda46b 100644
--- a/deps/jerry/CMakeLists.txt
+++ b/deps/jerry/CMakeLists.txt
@@ -137,7 +137,7 @@ macro(jerry_add_compile_warnings)
foreach(_warning ${ARGV})
jerry_add_compile_flags(-W${_warning})
if(USING_GCC)
- jerry_add_compile_flags(-Werror=${_warning})
+ jerry_add_compile_flags()
endif()
endforeach()
endmacro()
@@ -206,7 +206,7 @@ elseif(USING_CLANG)
endif()

if(JERRY_LIBC)
- jerry_add_compile_flags(-Werror)
+ jerry_add_compile_flags()
endif()

# C
1 change: 1 addition & 0 deletions debian/patches/series
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
0001-iotjs-Install-binaries-to-system-1134.patch
0002-build-Install-headers-to-system.patch
0003-cmake-Allow-one-to-select-python3-from-env.patch
0004-build-Disable-error-on-warning.patch

0 comments on commit c738b38

Please sign in to comment.