Re: [PATCH] BaseTools/Source/C/GenFv/GenFvInternalLib.c

["Gao, Liming" <liming.gao@intel.com>]

Yes. BaseTools Win32 Binary will auto be updated based on BaseTools source.

Thanks
Liming
From: Duran, Leo [mailto:leo.duran@amd.com]
Sent: Monday, August 15, 2016 9:13 PM
To: Zhu, Yonghong <yonghong.zhu@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Gao, Liming <liming.gao@intel.com>; edk2-devel@lists.01.org
Subject: RE: [edk2] [PATCH] BaseTools/Source/C/GenFv/GenFvInternalLib.c



> -----Original Message-----
> From: Zhu, Yonghong [mailto:yonghong.zhu@intel.com]
> Sent: Sunday, August 14, 2016 7:16 PM
> To: Ard Biesheuvel ; Duran, Leo
>
> Cc: Gao, Liming ; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>; Zhu,
> Yonghong
> Subject: RE: [edk2] [PATCH] BaseTools/Source/C/GenFv/GenFvInternalLib.c
>
> Leo, Thanks for the Contribution. It is good.
> Ard, Thanks for help to push the patch.
>
> Best Regards,
> Zhu Yonghong
[Duran, Leo]
Thank you Zhu and Ard.

BTW, I assume the executable 'auto-magically' gets built from source?
(I only submitted a patch for the source... Was that sufficient?)

Thanks,
Leo.

>
> -----Original Message-----
> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> Sent: Monday, August 15, 2016 3:25 AM
> To: Duran, Leo
> Cc: Gao, Liming ; Zhu, Yonghong
> ; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>
> Subject: Re: [edk2] [PATCH] BaseTools/Source/C/GenFv/GenFvInternalLib.c
>
> On 14 August 2016 at 17:05, Duran, Leo wrote:
> >
> >> -----Original Message-----
> >> From: Gao, Liming [mailto:liming.gao@intel.com]
> >> Sent: Friday, August 12, 2016 3:32 AM
> >> To: Duran, Leo ; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>
> >> Cc: Zhu, Yonghong
> >> Subject: RE: [PATCH] BaseTools/Source/C/GenFv/GenFvInternalLib.c
> >>
> >> Duran:
> >> Reusing FvForceRebase flag is a good idea to resolve this issue. I
> >> agree your change. Reviewed-by: Liming Gao
> >>
> > [Duran, Leo]
> > Excellent, thanks!
> >
> > Please advise on "next steps":
> > 1) Do we need a reply from Yonghong Zhu?
> > 2) When would it be reasonable to expect integration of this code into
> mainline?
> >
>
> Hi all,
>
> I pushed this (with Liming's R-b) as
>
> adb6ac256338 BaseTools/GenFv: Account for rebase of FV section containing
> VTF file
>
> Thanks,
> Ard.