On 05/04/19 15:35, Philippe Mathieu-Daudé wrote: > In commit 1cab464136b4 we incorrectly described the > EDK2_BASETOOLS_OPTFLAGS can pass CPPFLAGS and CFLAGS > options to the EDK2 build tools, but it only expands > the CFLAGS (not to the CPPFLAGS). > Update the description to be more accurate. > > Reported-by: Laszlo Ersek <ler...@redhat.com> > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> > --- > roms/Makefile | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/roms/Makefile b/roms/Makefile > index 0ce84a45ad5..f020102c866 100644 > --- a/roms/Makefile > +++ b/roms/Makefile > @@ -121,8 +121,8 @@ build-efi-roms: build-pxe-roms > $(patsubst %,bin-i386-efi/%.efidrv,$(pxerom_targets)) \ > $(patsubst %,bin-x86_64-efi/%.efidrv,$(pxerom_targets)) > > -# Build scripts can pass compiler/linker flags to the EDK2 build tools > -# via the EDK2_BASETOOLS_OPTFLAGS (CPPFLAGS and CFLAGS) and > +# Build scripts can pass compiler/linker flags to the EDK2 > +# build tools via the EDK2_BASETOOLS_OPTFLAGS (CFLAGS) and
please 'tr -s " "' in the above. With that: Reviewed-by: Laszlo Ersek <ler...@redhat.com> Thanks Laszlo > # EDK2_BASETOOLS_LDFLAGS (LDFLAGS) environment variables. > # > # Example: >
