Attention is currently required from: Arthur Heymans, Kapil Porwal, Nick Vaccaro, Paul Menzel, Subrata Banik.
Jon Murphy has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81051?usp=email )
Change subject: soc/intel/cmn/cse: Simplify logic to get CSE RW version ......................................................................
Patch Set 3:
(1 comment)
Patchset:
PS3:
could you please share the full name or email ID?
You're seeing hexdump in the docker image as an artifact of including bsdextrautils(https://github.com/coreboot/coreboot/blob/main/util/docker/coreboot-sdk/Dock...) as Paul pointed out previously, but it isn't explicitly called out as a dependency(https://github.com/coreboot/coreboot/blob/main/util/docker/coreboot-sdk/Dock...) or required installation. For what it's worth, I personally have never used the docker container to build(Downstream we also don't use the container and would need to ensure that hexdump is installed in the chroot). So I think it's worth explicitly adding it to the Dockerfile manifest and documentation showing that it's necessary to build coreboot. Without doing so, you have potential to break in the future when hexdump is no longer bundled with bsdextrautils by default or we remove that package all together.
We should aim to have the most seamless and stable experience for the developers when adding these types of dependencies.