[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:
> Patch Set 8: Code-Review+2
> > But how to match and replace "VBOOT" without touching "*_VBOOT_*"?
>
> Bill, probably the easiest way is to use a few different patterns.
> For example:
>
> s/CONFIG(VBOOT)/CONFIG(VBOOT_VERIFY_FIRMWARE)/
>   s/config VBOOT/config VBOOT_VERIFY_FIRMWARE/
>
> But anyways, if you want to leave it as VBOOT, I guess that's fine too.

The first one (with parenthesis boundaries) should be fine, but the second one may affect "config VBOOT_*", e.g. modify "config VBOOT_LIB" into "config VBOOT_VERIFY_FIRMWARE_LIB".

The second case needs an regexp matching "VBOOT" in "VBOOT$", "VBOOT," (in Makefile.inc), "VBOOT ", but not "VBOOT_*".