Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-debug-collector / c85df9cacee73d271d4184a2ee77fc6901a3671d
commitc85df9cacee73d271d4184a2ee77fc6901a3671d[log] [tgz]
authorWilliam de Abreu Pinho <williamdapinho@gmail.com>Mon Feb 02 09:11:37 2026 -0800
committerWilliam de Abreu Pinho <williamdapinho@gmail.com>Tue Feb 10 05:08:18 2026 -0800
tree91998c5d60be1dc4c1c2a34496a1da4db72ed00f
parent2b8c2f0d9e42068cbe2a6f96c52dcad0729b565f [diff]
dreport: fix shellcheck warnings in dreport

- SC2155: split declare and assign for EPOCHTIME
- SC2236: use -n instead of ! -z
- SC2254: quote case patterns
- SC2046/SC2086: quote variables and command substitutions
- SC2231: quote glob pattern in for loop
- SC2006: use $() instead of backticks for getopt
- SC2181: disable with TODO comments (getopt pattern, create_archive)
- SC2002: disable with comment (gendumpheader moved to
  openpower-debug-collector)
- Remove tools/dreport.d/dreport from .shellcheck-ignore

Tested:

Ran format-code.sh from openbmc-build-scripts.

Change-Id: I9abdc69d7fd0e2e2882571d9fd145ae10742c742
Signed-off-by: William de Abreu Pinho <williamdapinho@gmail.com>
2 files changed
tree: 91998c5d60be1dc4c1c2a34496a1da4db72ed00f
  1. dump-extensions/
  2. host-transport-extensions/
  3. service_files/
  4. subprojects/
  5. test/
  6. tools/
  7. .clang-format
  8. .clang-tidy
  9. .gitignore
  10. .shellcheck-ignore
  11. bmc_dump_entry.cpp
  12. bmc_dump_entry.hpp
  13. core_manager.cpp
  14. core_manager.hpp
  15. core_manager_main.cpp
  16. dump-extensions.hpp
  17. dump_entry.cpp
  18. dump_entry.hpp
  19. dump_manager.cpp
  20. dump_manager.hpp
  21. dump_manager_bmc.cpp
  22. dump_manager_bmc.hpp
  23. dump_manager_faultlog.cpp
  24. dump_manager_faultlog.hpp
  25. dump_manager_main.cpp
  26. dump_offload.cpp
  27. dump_offload.hpp
  28. dump_serialize.cpp
  29. dump_serialize.hpp
  30. dump_types.cpp.mako
  31. dump_types.hpp.mako
  32. dump_utils.cpp
  33. dump_utils.hpp
  34. elog_watch.cpp
  35. elog_watch.hpp
  36. example_dump_types.yaml
  37. example_errors_watch.yaml
  38. faultlog_dump_entry.cpp
  39. faultlog_dump_entry.hpp
  40. ffdc
  41. host_transport_exts.hpp
  42. LICENSE
  43. map_gen.py
  44. meson.build
  45. meson.options
  46. OWNERS
  47. ramoops_manager.cpp
  48. ramoops_manager.hpp
  49. ramoops_manager_main.cpp
  50. README.md
  51. watch.cpp
  52. watch.hpp
README.md

phosphor-debug-collector

Phosphor Debug Collector provides mechanisms to collect various log files and system parameters. Used to troubleshoot problems in OpenBMC based systems.

One such mechanism is dreport, a script that collects debug data and packages it into an archive file.

To Build

To build this package with meson, do the following steps:

    1. meson builddir
    2. ninja -C builddir

To clean the built files run ninja -C builddir clean.

To run unit tests

Tests can be run in the CI docker container, refer local-ci-build

or with an OpenBMC x86 sdk(see below for x86 steps).

meson -Dtests=enabled build
ninja -C build test