File: 0003-Increase-default-timeouts-for-linter-tests.patch

package info (click to toggle)
ros2-ament-lint 0.20.2-1
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid
  • size: 2,832 kB
  • sloc: python: 9,581; ansic: 2,394; xml: 777; cpp: 21; makefile: 4
file content (110 lines) | stat: -rw-r--r-- 4,364 bytes parent folder | download | duplicates (2)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
From: =?utf-8?q?Timo_R=C3=B6hling?= <roehling@debian.org>
Date: Sun, 16 Oct 2022 19:25:28 +0200
Subject: Increase default timeouts for linter tests

---
 ament_cmake_clang_tidy/cmake/ament_clang_tidy.cmake |  4 ++--
 ament_cmake_copyright/cmake/ament_copyright.cmake   |  4 ++--
 ament_cmake_cppcheck/cmake/ament_cppcheck.cmake     |  2 +-
 ament_cmake_cpplint/cmake/ament_cpplint.cmake       | 10 ++--------
 ament_cmake_pclint/cmake/ament_pclint.cmake         |  2 +-
 5 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/ament_cmake_clang_tidy/cmake/ament_clang_tidy.cmake b/ament_cmake_clang_tidy/cmake/ament_clang_tidy.cmake
index 95df6ab..c3abf64 100644
--- a/ament_cmake_clang_tidy/cmake/ament_clang_tidy.cmake
+++ b/ament_cmake_clang_tidy/cmake/ament_clang_tidy.cmake
@@ -29,7 +29,7 @@
 # :type CONFIG_FILE: string
 # :param ARGN: the files or directories to check
 # :type ARGN: list of strings
-# :param TIMEOUT: the test timeout in seconds, default: 300
+# :param TIMEOUT: the test timeout in seconds, default: 600
 # :type TIMEOUT: integer
 #
 # @public
@@ -68,7 +68,7 @@ function(ament_clang_tidy)
   endif()
 
   if(NOT ARG_TIMEOUT)
-    set(ARG_TIMEOUT 300)
+    set(ARG_TIMEOUT 600)
   endif()
 
   file(MAKE_DIRECTORY "${CMAKE_BINARY_DIR}/ament_clang_tidy")
diff --git a/ament_cmake_copyright/cmake/ament_copyright.cmake b/ament_cmake_copyright/cmake/ament_copyright.cmake
index 94dd190..1bbc571 100644
--- a/ament_cmake_copyright/cmake/ament_copyright.cmake
+++ b/ament_cmake_copyright/cmake/ament_copyright.cmake
@@ -17,7 +17,7 @@
 #
 # :param TESTNAME: the name of the test, default: "copyright"
 # :type TESTNAME: string
-# :param TIMEOUT: the test timeout in seconds, default: 120
+# :param TIMEOUT: the test timeout in seconds, default: 600
 # :type TIMEOUT: integer
 # :param EXCLUDE: an optional list of exclude files or directories for copyright check
 # :type EXCLUDE: list
@@ -32,7 +32,7 @@ function(ament_copyright)
     set(ARG_TESTNAME "copyright")
   endif()
   if(NOT ARG_TIMEOUT)
-    set(ARG_TIMEOUT 200)
+    set(ARG_TIMEOUT 600)
   endif()
 
   find_program(ament_copyright_BIN NAMES "ament_copyright")
diff --git a/ament_cmake_cppcheck/cmake/ament_cppcheck.cmake b/ament_cmake_cppcheck/cmake/ament_cppcheck.cmake
index cc66f41..0f2fa8a 100644
--- a/ament_cmake_cppcheck/cmake/ament_cppcheck.cmake
+++ b/ament_cmake_cppcheck/cmake/ament_cppcheck.cmake
@@ -66,7 +66,7 @@ function(ament_cppcheck)
   ament_add_test(
     "${ARG_TESTNAME}"
     COMMAND ${cmd}
-    TIMEOUT 300
+    TIMEOUT 600
     OUTPUT_FILE "${CMAKE_BINARY_DIR}/ament_cppcheck/${ARG_TESTNAME}.txt"
     RESULT_FILE "${result_file}"
     WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
diff --git a/ament_cmake_cpplint/cmake/ament_cpplint.cmake b/ament_cmake_cpplint/cmake/ament_cpplint.cmake
index 4398450..e2c9f5b 100644
--- a/ament_cmake_cpplint/cmake/ament_cpplint.cmake
+++ b/ament_cmake_cpplint/cmake/ament_cpplint.cmake
@@ -24,7 +24,7 @@
 # :type MAX_LINE_LENGTH: integer
 # :param ROOT: override the --root option of cpplint
 # :type ROOT: string
-# :param TIMEOUT: the test timeout in seconds, default: 120
+# :param TIMEOUT: the test timeout in seconds, default: 600
 # :type TIMEOUT: integer
 # :param EXCLUDE: an optional list of exclude directories or files for cpplint
 # :type EXCLUDE: list
@@ -61,13 +61,7 @@ function(ament_cpplint)
   endif()
   list(APPEND cmd ${ARG_UNPARSED_ARGUMENTS})
   if(NOT ARG_TIMEOUT)
-    if(WIN32)
-      # There are many timeouts when cpplint is executed on Windows,
-      # increasing the timeout seems to fix the problem.
-      set(ARG_TIMEOUT 500)
-    else()
-      set(ARG_TIMEOUT 120)
-    endif()
+    set(ARG_TIMEOUT 600)
   endif()
   if(NOT ARG_TIMEOUT GREATER 0)
     message(FATAL_ERROR "ament_add_test() the TIMEOUT argument must be a "
diff --git a/ament_cmake_pclint/cmake/ament_pclint.cmake b/ament_cmake_pclint/cmake/ament_pclint.cmake
index bdac4a4..927e18a 100644
--- a/ament_cmake_pclint/cmake/ament_pclint.cmake
+++ b/ament_cmake_pclint/cmake/ament_pclint.cmake
@@ -107,7 +107,7 @@ function(ament_pclint)
       OUTPUT_FILE "${CMAKE_BINARY_DIR}/ament_pclint/${ARG_TESTNAME}.txt"
       RESULT_FILE "${result_file}"
       WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
-      TIMEOUT 120)
+      TIMEOUT 600)
     set_tests_properties(
       "${ARG_TESTNAME}"
       PROPERTIES