Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-host-ipmid / deeceae31342652158460c502ee02c0b51cdd507
commitdeeceae31342652158460c502ee02c0b51cdd507[log] [tgz]
authorKonstantin Aladyshev <aladyshev22@gmail.com>Tue Jan 28 17:22:47 2025 +0300
committerVernon Mauery <vernon.mauery@linux.intel.com>Thu Feb 13 15:01:21 2025 +0000
treefca8756d51a031977c61612fee70684c7a07a49c
parent62ce1594fde138f073749e0bc56f6a4f1bb6b204 [diff]
Fix bug in the name of array of whitelisted commands

The commit 68d9d40530351e3f3db2c00a25969b0efc9fbe3f
("Update terminology used within repo") has changed the name of the
array of whitelisted IPMI commands from 'whitelist' to 'allowlist'
in the .hpp code, but didn't do it in the "generate_whitelist.sh"
script that generates the .cpp file with the relevant array.
This leads to the following error message at the service start:
"""
ipmid[363]: ERROR opening IPMI provider
/usr/lib/ipmid-providers/libwhitelist.so.0.1:
/usr/lib/ipmid-providers/libwhitelist.so.0.1: undefined symbol:
allowlist
"""
Correct array name to fix the issue.

Change-Id: I6d2fe94d0712181db13bb71cf2bd71e2db54d6c0
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
1 file changed
tree: fca8756d51a031977c61612fee70684c7a07a49c
  1. app/
  2. dbus-sdr/
  3. docs/
  4. include/
  5. libipmid/
  6. scripts/
  7. softoff/
  8. subprojects/
  9. test/
  10. user_channel/
  11. xyz/
  12. .build.sh
  13. .clang-format
  14. .gitignore
  15. .shellcheck
  16. .travis.yml
  17. apphandler.cpp
  18. apphandler.hpp
  19. chassishandler.cpp
  20. chassishandler.hpp
  21. dcmihandler.cpp
  22. dcmihandler.hpp
  23. error-HostEvent.hpp
  24. fruread.hpp
  25. generate_whitelist.sh
  26. generate_whitelist_create.sh
  27. globalhandler.cpp
  28. globalhandler.hpp
  29. groupext.cpp
  30. host-cmd-manager.cpp
  31. host-cmd-manager.hpp
  32. host-interface.cpp
  33. host-interface.hpp
  34. host-ipmid-whitelist.conf
  35. ipmi_fru_info_area.cpp
  36. ipmi_fru_info_area.hpp
  37. ipmiallowlist.hpp
  38. ipmid-new.cpp
  39. ipmisensor.cpp
  40. LICENSE
  41. meson.build
  42. meson.options
  43. OWNERS
  44. read_fru_data.cpp
  45. read_fru_data.hpp
  46. README.md
  47. selutility.cpp
  48. selutility.hpp
  49. sensordatahandler.cpp
  50. sensordatahandler.hpp
  51. sensorhandler.cpp
  52. sensorhandler.hpp
  53. settings.cpp
  54. settings.hpp
  55. storageaddsel.cpp
  56. storageaddsel.hpp
  57. storagehandler.cpp
  58. storagehandler.hpp
  59. sys_info_param.cpp
  60. sys_info_param.hpp
  61. systemintfcmds.cpp
  62. systemintfcmds.hpp
  63. transportconstants.hpp
  64. transporthandler.cpp
  65. transporthandler.hpp
  66. whitelist-filter.cpp
README.md

phosphor-host-ipmid

Compile ipmid with default options

meson builddir
ninja -C builddir

Compile ipmid with yocto defaults

meson builddir -Dbuildtype=minsize -Db_lto=true -Dtests=disabled
ninja -C builddir

If any of the dependencies are not found on the host system during configuration, meson automatically gets them via its wrap dependencies mentioned in ipmid/subprojects.

Enable/Disable meson wrap feature

meson builddir -Dwrap_mode=nofallback
ninja -C builddir

Enable debug traces

meson builddir -Dbuildtype=debug
ninja -C builddir

Generate test coverage report

meson builddir -Db_coverage=true -Dtests=enabled
ninja -C builddir test
ninja -C builddir coverage