unit-test-docker: replace valgrind-dbg

Get the following error with a clean run of this script:
base: Package valgrind-dbg is not available, but is referred to by
another package.
base: This m

unit-test-docker: replace valgrind-dbg

Get the following error with a clean run of this script:
base: Package valgrind-dbg is not available, but is referred to by
another package.
base: This may mean that the package is missing, has been obsoleted, or
base: is only available from another source
base: However the following packages replace it:
base: valgrind-mpi-dbgsym valgrind-dbgsym
base:
base: E: Package 'valgrind-dbg' has no installation candidate

It doesn't seem the valgrind-mpi package is needed for our use cases so
just add valgrind-dbgsym

Tested:
- Verified script now runs without error

Change-Id: Id0915c1dfa442dd02425ade14334e19538741067
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>

show more ...

Add libpeci to the build

libpeci has moved to meson and is used by other packages, so include it
in the CI build.

Tested:
Confirmed that peci-pcie CI build correctly pulls in libpeci.

Change-Id: I

Add libpeci to the build

libpeci has moved to meson and is used by other packages, so include it
in the CI build.

Tested:
Confirmed that peci-pcie CI build correctly pulls in libpeci.

Change-Id: I832260825fc13a614da0e95cf262a319a9e2ce35
Signed-off-by: Jason M. Bills <jason.m.bills@intel.com>

show more ...

fmt: Bump 9.1.0 -> 10.0.0

This matches the current yocto version.

Change-Id: I10f36903675cb32b3b4630e499dc0ccd72a64e4d
Signed-off-by: William A. Kennington III <wak@google.com>

build-unit-test-docker: patch meson for C++23 / clang

Meson 1.1.1 advertised support for C++23 but they only actually
implemented it for GCC. This is good enough for Bitbake builds
and many reposit

build-unit-test-docker: patch meson for C++23 / clang

Meson 1.1.1 advertised support for C++23 but they only actually
implemented it for GCC. This is good enough for Bitbake builds
and many repositories, but it is insufficient for any repository
that attempts to build in clang or run clang-tidy.

An upstream PR is available to fix this, but it is currently targeted
for Meson 1.3 ( mesonbuild/meson#11986 ).

In the meantime, use a patched version of Meson in order to enable
clang-based building with C++23 repositories.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I640e3f80c9484da5f73272d0004070cd252d03c9

show more ...

build-unit-test-docker: upgrade CLI11 to 2.3.2

The latest recipe in Yocto is pointing at a much newer release, so
update our Docker container also.

Signed-off-by: Patrick Williams <patrick@stwcx.xy

build-unit-test-docker: upgrade CLI11 to 2.3.2

The latest recipe in Yocto is pointing at a much newer release, so
update our Docker container also.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Iec7a7135862ff0ac5df9c964cade561543f765ac

show more ...

unit-test: add meson version check for C++23

We can start using C++23 in Meson, but it requires meson 1.1.1 or
later. Add the meson_version check in unit-test just like we've
done in the past for C

unit-test: add meson version check for C++23

We can start using C++23 in Meson, but it requires meson 1.1.1 or
later. Add the meson_version check in unit-test just like we've
done in the past for C++20.

Tested:

Changed sdbusplus to only have 'cpp_std=c++23' while using older
meson_version and experienced the expected exception.

```
Exception: C++23 support requires specifying in meson.build: meson_version: '>=1.1.1'
```

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I3d355f5176fb1874b7301ab995e96f22dbd6b429

show more ...

build-unit-test-docker: upgrade meson to 1.1.1

Yocto now includes Meson 1.1.1, which also happens to support
the `cpp_std=cpp23` option. Update the Docker container so
repositories could begin usin

build-unit-test-docker: upgrade meson to 1.1.1

Yocto now includes Meson 1.1.1, which also happens to support
the `cpp_std=cpp23` option. Update the Docker container so
repositories could begin using C++23.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I3388d3d872e1f43cf783214d55f03ec4ec05d9a8

show more ...

format-code: resolve symlinks in linter arguments

The latest version of prettier does not like being passed symbolic
links as arguments and refuses to follow the symlink. Add a call to
`realpath` w

format-code: resolve symlinks in linter arguments

The latest version of prettier does not like being passed symbolic
links as arguments and refuses to follow the symlink. Add a call to
`realpath` when we find all the lint-able files in order to resolve
the symlinks.

Tested:

The following error goes away:
```
[error] Explicitly specified pattern ".prettierrc.yaml" is a symbolic link.
[error] Explicitly specified pattern ".markdownlint.yaml" is a symbolic link.
```

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I18bac3e86253601140f034dc1b11f178443b6ed9

show more ...

Fix gpgv install fail

We expect use the gpgv and ubuntu-dbgsym-keyring default, without the
option `-yy`, the install command will fail default.

Change-Id: I70df0d90ec613b9d03871ba9509cacd30957d890

Fix gpgv install fail

We expect use the gpgv and ubuntu-dbgsym-keyring default, without the
option `-yy`, the install command will fail default.

Change-Id: I70df0d90ec613b9d03871ba9509cacd30957d890
Signed-off-by: Jian Zhang <zhangjian.3032@bytedance.com>

show more ...

build-unit-test-docker: upgrade googletest to 1.13.0

We've already incorporated a Yocto bump that moves to GTest 1.13,
so we might as well play along in the CI Docker container too.

Signed-off-by:

build-unit-test-docker: upgrade googletest to 1.13.0

We've already incorporated a Yocto bump that moves to GTest 1.13,
so we might as well play along in the CI Docker container too.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I44b19e24dc2641ef6e1cba6eea416b4578931c96

show more ...

Add nghttp2 v1.54.0 as a dependency

The new bmcweb http2 support[1] is using nghttp2. Add it as a
dependency.

Given there will likely only be one user, add it as:
1. Static library only (so unused

Add nghttp2 v1.54.0 as a dependency

The new bmcweb http2 support[1] is using nghttp2. Add it as a
dependency.

Given there will likely only be one user, add it as:
1. Static library only (so unused functions can be deleted at link time)
2. Only build the library (don't build examples or CLI applications).

[1] https://gerrit.openbmc.org/c/openbmc/bmcweb/+/40978

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I92d97be97a44f36693c97ba504037e73112a30c9

show more ...

build-unit-test-docker: Use `meson setup`

Invoking meson with only a build directory argument is deprecated:

```
2 10:14:32 andrew@fedora:~$ meson --version
1.1.0
2 10:17:32 andrew@fedora:~$ meson

build-unit-test-docker: Use `meson setup`

Invoking meson with only a build directory argument is deprecated:

```
2 10:14:32 andrew@fedora:~$ meson --version
1.1.0
2 10:17:32 andrew@fedora:~$ meson builddir
...
WARNING: Running the setup command as `meson [options]` instead of `meson setup [options]` is ambiguous and deprecated.
2 10:17:34 andrew@fedora:~$
```

Change-Id: I29c087cce5b4c2bc928a1cd4c6f2c0afa74a447f
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

build-unit-test-docker: Upgrade ubuntu lunar to mantic

Primarily this will use the stable version of gcc 13 instead of a git
version.

Change-Id: I915f5456e7b82904368841d94120be03ac38987a
Signed-off

build-unit-test-docker: Upgrade ubuntu lunar to mantic

Primarily this will use the stable version of gcc 13 instead of a git
version.

Change-Id: I915f5456e7b82904368841d94120be03ac38987a
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

build-unit-test-docker: Build libpldm stable ABI

Don't include any symbols that are still marked as 'testing' in the
libpldm ABI. No applications should pass CI while relying on symbols
that aren't

build-unit-test-docker: Build libpldm stable ABI

Don't include any symbols that are still marked as 'testing' in the
libpldm ABI. No applications should pass CI while relying on symbols
that aren't stable.

See `libpldm: Explicit deprecated, stable and testing ABI classes`[1].

[1]: https://gerrit.openbmc.org/c/openbmc/libpldm/+/63974

Change-Id: Ia99c512566a9d3723e772d64c10d6f82a73b48af
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

build-unit-test-docker: update to gcc-13

Yocto upstream is now on GCC-13, which we will be merging as soon as all
code compiles successfully. Update the docker container for CI as well
so we do not

build-unit-test-docker: update to gcc-13

Yocto upstream is now on GCC-13, which we will be merging as soon as all
code compiles successfully. Update the docker container for CI as well
so we do not get any regressions.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I8bab7fb5acfe0d19d03729a69d8cacac7e571b5b

show more ...

unit-test: Increase available debug information for test builds

Configuring `-Og` helps ensure `abi-dumper` has all the information it
would like for ABI analysis.

```
...
[18/39] Generatin

unit-test: Increase available debug information for test builds

Configuring `-Og` helps ensure `abi-dumper` has all the information it
would like for ABI analysis.

```
...
[18/39] Generating abi-dump with a custom command
Detect public symbols
Reading debug-info
WARNING: incompatible build option detected: -O0 (required -Og for better analysis)
...
```

Change-Id: Ia4d2add17a1e65bb480cb7003597af62c8e47bc2
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

build-unit-test-docker: Also install abi-dumper

It turns out `abi-dumper` is not a dependency of
`abi-compliance-checker`, so we must install both.

Change-Id: I37547f87106cb738d155e53108e581c0e1395

build-unit-test-docker: Also install abi-dumper

It turns out `abi-dumper` is not a dependency of
`abi-compliance-checker`, so we must install both.

Change-Id: I37547f87106cb738d155e53108e581c0e13958f3
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

build-unit-test-docker: Install abi-compliance-checker

abi-compliance-checker enables enforcement of ABI/API compliance in CI
for projects producing shared libraries. A reference implementation
exis

build-unit-test-docker: Install abi-compliance-checker

abi-compliance-checker enables enforcement of ABI/API compliance in CI
for projects producing shared libraries. A reference implementation
exists for libpldm:

https://gerrit.openbmc.org/c/openbmc/libpldm/+/63654

Change-Id: I82071e4d2bff7307dd6b8269efed04cac0f13959
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...

build-unit-test-docker: use curl

curl has better proxy support than wget - for example socks5h.

Change-Id: I221295a9a0ceabc0370d8d6d3cd4a4edd8438988
Signed-off-by: Brad Bishop <bradleyb@fuzziesquir

build-unit-test-docker: use curl

curl has better proxy support than wget - for example socks5h.

Change-Id: I221295a9a0ceabc0370d8d6d3cd4a4edd8438988
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

show more ...

build-unit-test-docker: workaround boost and clang-16 issues

Add a patch to work around the issue discussed in boostorg/beast#2648
and observed in bmcweb testing. This patch is only necessary in ou

build-unit-test-docker: workaround boost and clang-16 issues

Add a patch to work around the issue discussed in boostorg/beast#2648
and observed in bmcweb testing. This patch is only necessary in our
Docker container, and not Yocto, because we only compile with clang
or clang-tidy in the Docker environment.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I9e68c74fd8ee1421bf9e49f9f6ce497c8b05a436

show more ...

build-unit-test-docker: update to clang-16

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I59cf51cdb89476d9da4934d0cf6be44fc2cf6bc1

Update boost 1.81 -> 1.82

The latest yocto update pulled in the new boost version. Update it.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Idb39a2ac496b6c1e54cd6c87682c7847e7f848a8

build-unit-test-docker: move homedir with usermod

When the uid already exists, we do a usermod instead of a useradd,
but this does not create the new home directory. We need to pass
the `-m` argume

build-unit-test-docker: move homedir with usermod

When the uid already exists, we do a usermod instead of a useradd,
but this does not create the new home directory. We need to pass
the `-m` argument in order to move the old home directory to the
new location.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I9f2cabe982af0b09d3c3f65dcd3a0acb6b8f3591

show more ...

build-unit-test-docker: Use curl for getting keys

apt-key fails to get keys when behind a firewall. Use curl to
get keys via http proxy and add them using apt-key add.

Tested: docker build and run

build-unit-test-docker: Use curl for getting keys

apt-key fails to get keys when behind a firewall. Use curl to
get keys via http proxy and add them using apt-key add.

Tested: docker build and run using "scripts/build-unit-test-docker".

Change-Id: I3fa3e4cdd326681b72a77ac251de76a9359ce706
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>

show more ...

unit-test: Make UBSAN kill tests

Continuing to exploit the runtime library means we get useful
diagnostics printed rather than just the application generating a core.

Change-Id: I0b93e61bbca48eb5a7

unit-test: Make UBSAN kill tests

Continuing to exploit the runtime library means we get useful
diagnostics printed rather than just the application generating a core.

Change-Id: I0b93e61bbca48eb5a79ba6fa9228431d36504f64
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

show more ...