1 |
commit: 76b20521901742c59aa823035357a3a49ebb0aad |
2 |
Author: François-Xavier Carton <fx.carton91 <AT> gmail <DOT> com> |
3 |
AuthorDate: Thu Sep 29 18:08:30 2022 +0000 |
4 |
Commit: Andrew Ammerlaan <andrewammerlaan <AT> gentoo <DOT> org> |
5 |
CommitDate: Fri Sep 30 06:49:55 2022 +0000 |
6 |
URL: https://gitweb.gentoo.org/proj/sci.git/commit/?id=76b20521 |
7 |
|
8 |
sci-libs/itk: add upstream patch to fix building tests |
9 |
|
10 |
Needed since commit b6218b "sci-libs/itk: add AdaptiveDenoising module" |
11 |
because the commit used for that remote module is too recent and needs |
12 |
that ITK commit. |
13 |
|
14 |
Signed-off-by: François-Xavier Carton <fx.carton91 <AT> gmail.com> |
15 |
Closes: https://github.com/gentoo/sci/pull/1172 |
16 |
Signed-off-by: Andrew Ammerlaan <andrewammerlaan <AT> gentoo.org> |
17 |
|
18 |
sci-libs/itk/files/itk-5.2.1-upstream-fixes.patch | 54 ++++++++++++++++++++++ |
19 |
.../{itk-5.2.1-r1.ebuild => itk-5.2.1-r2.ebuild} | 0 |
20 |
2 files changed, 54 insertions(+) |
21 |
|
22 |
diff --git a/sci-libs/itk/files/itk-5.2.1-upstream-fixes.patch b/sci-libs/itk/files/itk-5.2.1-upstream-fixes.patch |
23 |
index 2e547d9ca..2af8c4998 100644 |
24 |
--- a/sci-libs/itk/files/itk-5.2.1-upstream-fixes.patch |
25 |
+++ b/sci-libs/itk/files/itk-5.2.1-upstream-fixes.patch |
26 |
@@ -1,6 +1,7 @@ |
27 |
https://github.com/InsightSoftwareConsortium/ITK/commit/2a7f5ed199926999b67f61a97c2c9a2595a11558.patch |
28 |
https://github.com/InsightSoftwareConsortium/ITK/commit/0c31aa688e82a34e5430873bc6176740fd09ad75.patch |
29 |
https://github.com/InsightSoftwareConsortium/ITK/commit/28325d921a9b8793e6aaefd612f53989f951d19c.patch |
30 |
+https://github.com/InsightSoftwareConsortium/ITK/commit/aafd1f9631d0fad60e1b6f53e049e95a50c48bd1.patch |
31 |
|
32 |
From 2a7f5ed199926999b67f61a97c2c9a2595a11558 Mon Sep 17 00:00:00 2001 |
33 |
From: Niels Dekker <N.Dekker@××××.nl> |
34 |
@@ -253,3 +254,56 @@ index fca3d999b87..3b7a1d37334 100644 |
35 |
file(WRITE ${CMAKE_CURRENT_BINARY_DIR}/detect_site_package_path.py ${python_check}) |
36 |
execute_process(COMMAND "${Python3_EXECUTABLE}" "${CMAKE_CURRENT_BINARY_DIR}/detect_site_package_path.py" |
37 |
OUTPUT_VARIABLE py_spp |
38 |
+From aafd1f9631d0fad60e1b6f53e049e95a50c48bd1 Mon Sep 17 00:00:00 2001 |
39 |
+From: Hans Johnson <hans-johnson@×××××.edu> |
40 |
+Date: Wed, 15 Dec 2021 08:10:58 -0600 |
41 |
+Subject: [PATCH] COMP: Remove inclusion of .hxx files as headers |
42 |
+ |
43 |
+The ability to include either .h or .hxx files as |
44 |
+header files required recursively reading the |
45 |
+.h files twice. The added complexity is |
46 |
+unnecessary, costly, and can confuse static |
47 |
+analysis tools that monitor header guardes (due |
48 |
+to reaching the maximum depth of recursion |
49 |
+limits for nested #ifdefs in checking). |
50 |
+ |
51 |
+Co-authored-by: Simon Rit <simon.rit@×××××××××××××××××.fr> |
52 |
+--- |
53 |
+ CMake/ITKModuleHeaderTest.cmake | 3 +-- |
54 |
+ Utilities/Maintenance/BuildHeaderTest.py | 4 ---- |
55 |
+ 2 files changed, 1 insertion(+), 6 deletions(-) |
56 |
+ |
57 |
+diff --git a/CMake/ITKModuleHeaderTest.cmake b/CMake/ITKModuleHeaderTest.cmake |
58 |
+index eb809d7e93d..82ad192a6f8 100644 |
59 |
+--- a/CMake/ITKModuleHeaderTest.cmake |
60 |
++++ b/CMake/ITKModuleHeaderTest.cmake |
61 |
+@@ -36,8 +36,7 @@ macro( itk_module_headertest _name ) |
62 |
+ # effected. |
63 |
+ set( _include ${${_name}_SOURCE_DIR}/include ) |
64 |
+ file( GLOB _h_files ${_include}/*.h ) |
65 |
+- file( GLOB _hxx_files ${_include}/*.hxx ) |
66 |
+- set( _header_files ${_h_files} ${_hxx_files} ) |
67 |
++ set( _header_files ${_h_files} ) |
68 |
+ list( LENGTH _h_files _num_headers ) |
69 |
+ set( _outputs ${${_name}_BINARY_DIR}/test/${_name}HeaderTest1.cxx ) |
70 |
+ set( _test_num 1 ) |
71 |
+diff --git a/Utilities/Maintenance/BuildHeaderTest.py b/Utilities/Maintenance/BuildHeaderTest.py |
72 |
+index 26a4b6ccb3f..f8ed9f7901e 100755 |
73 |
+--- a/Utilities/Maintenance/BuildHeaderTest.py |
74 |
++++ b/Utilities/Maintenance/BuildHeaderTest.py |
75 |
+@@ -130,15 +130,11 @@ def main(): |
76 |
+ else: |
77 |
+ max_idx = added_header_idx + maximum_number_of_headers |
78 |
+ for i in range(added_header_idx, max_idx): |
79 |
+- # Use the .hxx if possible. |
80 |
+- hxx_file = h_files[i][:-1] + "hxx" |
81 |
+ # Files that include VTK headers need to link to VTK. |
82 |
+ if h_files[i] in BANNED_HEADERS or h_files[i].lower().find("vtk") != -1: |
83 |
+ to_include = ( |
84 |
+ '// #include "' + h_files[i] + '" // Banned in BuildHeaderTest.py\n' |
85 |
+ ) |
86 |
+- elif os.path.exists(os.path.join(module_source_path, "include", hxx_file)): |
87 |
+- to_include = '#include "' + hxx_file + '"\n' |
88 |
+ else: |
89 |
+ to_include = '#include "' + h_files[i] + '"\n' |
90 |
+ |
91 |
|
92 |
diff --git a/sci-libs/itk/itk-5.2.1-r1.ebuild b/sci-libs/itk/itk-5.2.1-r2.ebuild |
93 |
similarity index 100% |
94 |
rename from sci-libs/itk/itk-5.2.1-r1.ebuild |
95 |
rename to sci-libs/itk/itk-5.2.1-r2.ebuild |