Skip to content

Commit f914067

Browse files
authored
Fix-build-issues-clang-13 (#291)
* Reorder includes in deferred.cpp * fix msan complaining about system dependencies * Print config log after configure * print config.log on failure * Temporarily run msan on ubuntu 20.04 and clang-6 * Run msan on 18.04 * Fix clang package name to clang-6.0
1 parent ea900cd commit f914067

File tree

3 files changed

+14
-4
lines changed

3 files changed

+14
-4
lines changed

.github/workflows/verify-build.yml

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,12 +63,12 @@ jobs:
6363
debug: debug
6464
coverage: nocoverage
6565
- test-group: extra
66-
os: ubuntu-latest
66+
os: ubuntu-18.04
6767
os-type: ubuntu
6868
build-type: msan
6969
compiler-family: clang
70-
c-compiler: clang-13
71-
cc-compiler: clang++-13
70+
c-compiler: clang-6.0
71+
cc-compiler: clang++-6.0
7272
debug: debug
7373
coverage: nocoverage
7474
- test-group: extra
@@ -442,6 +442,12 @@ jobs:
442442
../configure --disable-fastopen;
443443
fi
444444
445+
- name: Print config.log
446+
run: |
447+
cd build ;
448+
cat config.log ;
449+
if: ${{ failure() }}
450+
445451
# Make or run iwyu. If running iwyu, check for the result code to be 2 (IWYU always returns an error code, if it is 2, no corrections are necessary).
446452
- name: Make or run IWYU
447453
run: |

msan_ignorelist.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
fun:__interceptor_strlen
2+
fun:__interceptor_fopen64
3+
fun:__interceptor_memcmp

test/integ/deferred.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,12 @@
3030
#include <sys/socket.h>
3131
#endif
3232

33-
#include <cstring>
3433
#include <curl/curl.h>
3534
#include <signal.h>
3635
#include <unistd.h>
3736

37+
#include <cstring>
38+
3839
#include "./httpserver.hpp"
3940
#include "./littletest.hpp"
4041

0 commit comments

Comments
 (0)