Skip to content

Commit

Permalink
Merge pull request #17230 from branfosj/20230128121834_new_pr_NECI202…
Browse files Browse the repository at this point in the history
…20711

fix CMake print summary for long hostnames for NECI
  • Loading branch information
boegel authored Jan 29, 2023
2 parents 257857c + 165bb81 commit cb95a4e
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 1 deletion.
6 changes: 5 additions & 1 deletion easybuild/easyconfigs/n/NECI/NECI-20220711-foss-2022a.eb
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,11 @@ sources = [{
},
'filename': SOURCE_TAR_GZ,
}]
checksums = [None]
patches = ['NECI-20220711_fix-cmake-print-summary.patch']
checksums = [
None,
'e189f1b3991b28502dbd8285a04784e7e422d0a9e01e3f28025d0458b479af2d',
]

builddependencies = [
('CMake', '3.23.1'),
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
If the _content passed to the box_side macro is too long then it fails
Patch by Simon Branford (University of Birmingham)
--- cmake/neci_print_summary.cmake.orig 2023-01-28 12:12:05.844153000 +0000
+++ cmake/neci_print_summary.cmake 2023-01-28 12:12:26.698259000 +0000
@@ -28,9 +28,7 @@
feature_summary( WHAT ALL INCLUDE_QUIET_PACKAGES)

macro( box_side _content )
- string( LENGTH "${_content}" _len )
- string( SUBSTRING " |" ${_len} -1 _box_tail )
- message( STATUS " | ${_content}${_box_tail}" )
+ message( STATUS " | ${_content} |" )
endmacro()

string(TIMESTAMP _time "%Y-%m-%d %H:%M")

0 comments on commit cb95a4e

Please sign in to comment.