cossacklabs / themis / #7657
88%
master: 83%

LAST BUILD BRANCH: release/0.15.0
DEFAULT BRANCH: master
Ran
Jobs 1
Files 73
Run time 9s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 87.692%. Remained the same
#7657

push

travis-ci

web-flow 
Fix assorted issues with legacy systems (#644)

* Avoid inline variables in "for" loops

Some systems (like CentOS 7) still run their C compilers in some C89
compatibility mode, so any C99 code will fail to build there.

Since we do not require C99, declare variables at the start of blocks.

Note that we do use some other features like C++ comments so we cannot
enable strict C89 compatibility mode. CentOS 7 uses "-std=gnu90" which
includes various GNU extensions such as line comments, but not variable
declarations in arbitrary places.

* Do not use --no-undefined with Emscripten

It turns out that not all Emscripten toolchains support "--no-undefined"
flag and complain about that by breaking builds, in particular on our
Buildbot infrastructure.

We use "--no-undefined" to get warnings about missing dependencies which
are usually caused by OpenSSL header files that do not match the library
binaries actually linked. We use BoringSSL with Emscripten by default
so we don't need those warnings that much here.

* Drop layout tests from libthemis-sys

struct secure_session_user_callbacks_type and others include
architecture-specific types such as pointers and size_t values.

Commit 85af22d (Drop "bindgen" from libthemis-sys dependencies,
2020-04-25) has replaced on-the-fly invocation of Bindgen with
using pregenerated files. Unfortunately, we cannot use layout
tests in these files since they end up architecture-specific.

On 64-bit machines -- such as the one used to generate the files --
secure_session_user_callbacks_type uses 40 bytes of storage while
on 32-bit machiens is requires only 20 bytes. We cannot include
a test that simply compares the size with a constant "40".

I don't want to complicate things too much just because of those tests
so let's simply drop them if *they* are the only platform-dependent part
here.

* Ignore panicking corruption test in RustThemis

This test triggers a panic in "Vec::reserve... (continued)

3776 of 4306 relevant lines covered (87.69%)

19297.03 hits per line

Jobs
ID Job ID Ran Files Coverage
1 #7657.1 0
87.69
Source Files on build #7657
Detailed source file information is not available for this build.