Update language in ipmbbridge to be more inclusive

Use more inclusive terminology inside of ipmbbridge repo.

Tested: No functional change. Verified on Intel system that ipmb.service
runs and Ipmb c

Update language in ipmbbridge to be more inclusive

Use more inclusive terminology inside of ipmbbridge repo.

Tested: No functional change. Verified on Intel system that ipmb.service
runs and Ipmb channel still gets published on DBus.

Change-Id: I70577a3b28e209a38bcd3d2ec99dd17b09a2d723
Signed-off-by: Matt Simmering <matthew.simmering@intel.com>

show more ...

meson: Support building boost as subproject

Add support for the local meson build on the system without boost.

Tested:
"meson setup build && cd build && meson compile" finishes successfully.

Chang

meson: Support building boost as subproject

Add support for the local meson build on the system without boost.

Tested:
"meson setup build && cd build && meson compile" finishes successfully.

Change-Id: I22eb23c216f9333589ab61ae4d1cb707be5c9a0a
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>

show more ...

meson: adjust nlohmann-json dependency

Simplify the nlohmann-json dependency and align the wrap file with other
repositories.

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

meson: adjust nlohmann-json dependency

Simplify the nlohmann-json dependency and align the wrap file with other
repositories.

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

show more ...

clang-format: copy latest and re-format

clang-format-17 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-17 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

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

show more ...

build: upgrade to C++23

Meson 1.1.1 and GCC-13 both support C++23 and a sufficient portion of
the standard has been implemented. Upgrade the build to leverage it.

Change-Id: I9ea3416a29d8c921871c0

build: upgrade to C++23

Meson 1.1.1 and GCC-13 both support C++23 and a sufficient portion of
the standard has been implemented. Upgrade the build to leverage it.

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

show more ...

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

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

show more ...

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig` keyword argument. Ensure meson 0.58 is required and update
the usage of all `get_pkgconfig_variable` and `get_variable` to be the
modern variant.

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

show more ...

The executable was renamed to ipmbbridge

- The executable was renamed ipmbbridge while deleting CMake support, but ipmb.service expected it to be ipmbbridged.

ipmb.service: Failed to locate exe

The executable was renamed to ipmbbridge

- The executable was renamed ipmbbridge while deleting CMake support, but ipmb.service expected it to be ipmbbridged.

ipmb.service: Failed to locate executable /usr/bin/ipmbbridged: No such file or directory
ipmb.service: Failed at step EXEC spawning /usr/bin/ipmbbridged: No such file or directory
ipmb.service: Main process exited, code=exited, status=203/EXEC
ipmb.service: Failed with result 'exit-code'.

Test:
$ systemctl status ipmb
* ipmb.service - IPMB bridge
Loaded: loaded (/lib/systemd/system/ipmb.service; enabled; preset: enabled)
Active: active (running) since Fri 2023-03-03 09:49:51 UTC; 1 month 4 days ago
Main PID: 476 (ipmbbridged)
CPU: 599ms
CGroup: /system.slice/ipmb.service
`-476 /usr/bin/ipmbbridged

Change-Id: Ifc3b3525db9628ad76fa6f56b8817c7b0a332ac9
Signed-off-by: Jorge Cisneros <jcisneros3@lenovo.com>

show more ...

build: delete CMake support

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

build: add meson and subproject wrap files

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

squash pedantic warnings over flexible array members

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

clean up missing-field-initializers warnings

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

clean up unused parameter warnings

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

fix different signedness comparison

../ipmbbridged.cpp:368:12: warning: comparison of integer expressions of different signedness: ‘int’ and ‘const size_t’ {aka ‘const long unsigned int’} [-Wsign-co

fix different signedness comparison

../ipmbbridged.cpp:368:12: warning: comparison of integer expressions of different signedness: ‘int’ and ‘const size_t’ {aka ‘const long unsigned int’} [-Wsign-compare]

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

show more ...

simplify sequence number processing

Newer GCC didn't like the complexity of the sequence calculation,
so simplify it.

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

simplify sequence number processing

Newer GCC didn't like the complexity of the sequence calculation,
so simplify it.

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

show more ...

clang-format: update with latest

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

Add missing include

list was included through a dependency on sdbusplus which no longer
provides it, so it needs to be explicitly included here.

Change-Id: Iad2862e06fd31a95b9bbecb7246504833bc5f42d

Add missing include

list was included through a dependency on sdbusplus which no longer
provides it, so it needs to be explicitly included here.

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

show more ...

Change io_service to io_context

This was renamed a while back in boost to be compliant with the
std::executors proposal.

Change-Id: I2bda7bf0e3e010a4162073b579bb744901597062
Signed-off-by: Ed Tanou

Change io_service to io_context

This was renamed a while back in boost to be compliant with the
std::executors proposal.

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

show more ...

prettier: re-format

Prettier is enabled in openbmc-build-scripts on Markdown, JSON, and YAML
files to have consistent formatting for these file types. Re-run the
formatter on the whole repository.

prettier: re-format

Prettier is enabled in openbmc-build-scripts on Markdown, JSON, and YAML
files to have consistent formatting for these file types. Re-run the
formatter on the whole repository.

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

show more ...

Remove Dawid from the maintainers list

Dawid has not been involved at Intel for quite a while, his email
address bounces, and has not participated in the project in years.
Remove him from the mainta

Remove Dawid from the maintainers list

Dawid has not been involved at Intel for quite a while, his email
address bounces, and has not participated in the project in years.
Remove him from the maintainers list.

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

show more ...

Upgrade to c++20

C++20 is the standard on the project these days. Upgrade to it.

Tested: Code compiles.

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

Upgrade to c++20

C++20 is the standard on the project these days. Upgrade to it.

Tested: Code compiles.

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

show more ...

MAINTAINERS: remove file

The MAINTAINERS file is deprecated in favor of OWNERS.

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

OWNERS: switch 'matches' to 'matchers'

The original OWNERS template had a mistake which used 'matches' instead
of the field supported by the Gerrit plugin 'matchers'. Update the
OWNERS file to have

OWNERS: switch 'matches' to 'matchers'

The original OWNERS template had a mistake which used 'matches' instead
of the field supported by the Gerrit plugin 'matchers'. Update the
OWNERS file to have the correct field.

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

show more ...

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are for:
* bus_t
* exception_t
* manager_t
* match_t
* message_t
* object_t
* slot_t

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

show more ...

Add OWNERS file

Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
Change-Id: Ie69abcea58a51ba12b4736989742817a3ced26ad