[coreboot-gerrit] Change in coreboot[master]: security/vboot: Add a dedicated flag for building of vboot library

[Bill XIE (Code Review)]

Bill XIE has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37787 )

Change subject: security/vboot: Add a dedicated flag for building of vboot library
......................................................................

Patch Set 8:

(1 comment)
> Patch Set 7:
>
> (3 comments)
> > > Another naming option would be to use:
> > >
> > > VBOOT (or VBOOT_LIB) - include the vboot library functions
> > > VBOOT_VERIFY_FIRMWARE - enable firmware verification
> > >
> > > What do you think?
> >
> > I would rather keep the VBOOT flag, for I don't know how to efficiently and correctly change hundreds of its appearance around the whole coreboot tree.
>
> Check out the `sed` utility.

But how to match and replace "VBOOT" without touching "*_VBOOT_*"?

https://review.coreboot.org/c/coreboot/+/37787/7/src/security/vboot/Kconfig 
File src/security/vboot/Kconfig:

https://review.coreboot.org/c/coreboot/+/37787/7/src/security/vboot/Kconfig@... 
PS7, Line 22: Building and linking the vboot library.
> How about just "Build and link the vboot library. […]

Done