clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda forma

clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda formatting also changed, so we have made changes to the
organization default style format to better handle lambda formatting.

See I5e08687e696dd240402a2780158664b7113def0e for updated style.
See Iea0776aaa7edd483fa395e23de25ebf5a6288f71 for clang-18 enablement.

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

show more ...

Fix spelling mistakes using codespell

This commit corrects various spelling mistakes throughout the
repository. The corrections were made automatically using `codespell`[1]
tool.

[1]: https://githu

Fix spelling mistakes using codespell

This commit corrects various spelling mistakes throughout the
repository. The corrections were made automatically using `codespell`[1]
tool.

[1]: https://github.com/codespell-project/codespell

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

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: I3a511d2180b4f77d727275df11a4804d13bce05d
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

meson_options.txt: Support for reading options from meson.options

Support has been added for reading options from meson.options instead
of meson_options.txt[1]. These are equivalent, but not using t

meson_options.txt: Support for reading options from meson.options

Support has been added for reading options from meson.options instead
of meson_options.txt[1]. These are equivalent, but not using the .txt
extension for a build file has a few advantages, chief among them
many tools and text editors expect a file with the .txt extension to
be plain text files, not build scripts.

[1] https://mesonbuild.com/Release-notes-for-1-1-0.html#support-for-reading-options-from-mesonoptions

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I9afbcdacb8d77eed50119c015d71a6cdf0518d2b

show more ...

scripts: remove unused gen_pimrules.py

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

build: remove autotools support

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

build: install strgfnhandler into ipmi-providers dir

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

build: enable meson builds

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

scripts: accept absolute paths to YAML

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

frup: suppress stringop-truncation warning

The code involved here doesn't entirely make sense because it is
performing a sequence of hex to ascii conversions, but then
overwriting the beginning with

frup: suppress stringop-truncation warning

The code involved here doesn't entirely make sense because it is
performing a sequence of hex to ascii conversions, but then
overwriting the beginning with a "0x", which seems to lose the
first 2 bytes.

Until someone has a chance to unravel this code and turn it into
a more modern implementation, suppress the GCC warning about string
truncation (since the string truncation is done by design).

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

show more ...

strgfnhandler: suppress unused parameter warnings

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

writefrudata: remove vla

Variable length arrays are not allowed in standard C++. Remove them
in favor of std::vector.

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

writefrudata: remove vla

Variable length arrays are not allowed in standard C++. Remove them
in favor of std::vector.

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

show more ...

fru-parser: Remove the use of mktime

mktime get the time in localtime and will mess up the time if the
timezone != UTC. The FRU data should be the raw data in UTC and the
reader will convert to loca

fru-parser: Remove the use of mktime

mktime get the time in localtime and will mess up the time if the
timezone != UTC. The FRU data should be the raw data in UTC and the
reader will convert to localtime at their end.

This issue is detected when we change the timezone to PST/PDT with
https://gerrit.openbmc.org/c/openbmc/openbmc/+/58293
and the FRU EEPROM time does not match the expected Mfg Date anymore.

Used the timestamp of 1/1/1996 UTC directly.

Data Reader:
https://gerrit.openbmc.org/c/openbmc/phosphor-host-ipmid/+/58466

Tested:
Fru EEPROM Mfg Time now is in the same (after timezone conversion)
between the raw data and FRU output.

Change-Id: I9d2d045f037d1976e45ed4e0c1857bb8d5ebc06d
Signed-off-by: Willy Tu <wltu@google.com>

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: I59041b73cc8b15c2a3800911540e1a10011f6150
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

Set new maintainer as Patrick Williams

Remove Vernon Mauery as maintainer.

Change-Id: Ia0f4a860d7191079c3b0119789387ff28ea3712e
Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>

markdownlint: fix all warnings

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

python: fix flake8 warnings

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

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

Change-Id: I5e068068ef3faa2

black: re-format

black and isort are enabled in the openbmc-build-scripts on Python files
to have a consistent formatting. Re-run the formatter on the whole
repository.

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

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: I4e8e80d3e90345e107038546c485510b565ac636
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

build: remove libmapper dependency

Nothing in the codebase actually uses libmapper. Remove the
library dependency.

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

build: remove libmapper dependency

Nothing in the codebase actually uses libmapper. Remove the
library dependency.

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

show more ...

Fix offset size

Signed-off-by: Jean-Marie Verdun <jean-marie.verdun@hpe.com>
Change-Id: Ic6128e836c3add71737022b9bfb4c96713f0772c

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: Idab99ed8fdb9bdf5cc0e130d576f55d256df9cce

show more ...

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: I7450b52603a9f3d007c4685f1a3820e861610c31

show more ...

MAINTAINERS: remove file

The MAINTAINERS file is deprecated in favor of OWNERS.

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

build: enable C++20

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