On Fri, Jul 25, 2014 at 1:47 PM, Kinney, Michael D <[email protected]> wrote: > Jordan, > > I agree with doing these as 5 separate patches. > > Reviewed-by: Michael Kinney [email protected]
I'll add your r-b on patch 5, but I'll just upload the others using the original commit info. > Do you want to go ahead and check-in these patches? Sure. -Jordan > -----Original Message----- > From: Jordan Justen [mailto:[email protected]] > Sent: Friday, July 25, 2014 1:43 PM > To: Kinney, Michael D > Cc: [email protected]; > [email protected] > Subject: Re: [edk2-buildtools] [edk2][PATCH] Merge sources from > edk2-buildtools sub-project to EDK II BaseTools trunk > > Mike, > > I think we should re-apply these 4 changes separately in EDK II. (See > attachments 1-4) > > I also found a difference in some 'build version' variables. (See attachment > 5). > > After applying these 5 patches, the two BaseTools seem to be in sync. > > -Jordan > > https://github.com/jljusten/edk2/tree/sync-BaseTools > > On Fri, Jul 25, 2014 at 10:38 AM, Kinney, Michael D > <[email protected]> wrote: >> Hello, >> >> >> >> The attached patch contains the changes to sync the edk2-buildtools >> sub-project to the EDK II BaseTools trunk. This is the last step before the >> edk2-buildtools sub-project permissions are changed to read-only. >> >> >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> >> Signed-off-by: Michael Kinney [email protected] >> >> >> >> The patch includes changes for the following check-in log entries from the >> edk2-buildtools sub-project. >> >> >> >> ===== >> >> Update GenFv tool to handle the file path with space. >> >> >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> >> Signed-off-by: Gao, Liming [email protected] >> >> Review-by: Kinney, Michael D [email protected] >> >> ==== >> >> Report warning info if vfr file use the flags used for framework code only. >> >> >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> >> Signed-off-by: Eric Dong <[email protected]> >> >> Reviewed-by: Liming Gao <[email protected]> >> >> ==== >> >> Clean 1 build error and 1 warning for VfrCompiler. >> >> >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> >> Signed-off-by: Eric Dong <[email protected]> >> >> Reviewed-by: Liming Gao <[email protected]> >> >> ==== >> >> Report warning info if an action opcode has text two statement. >> >> >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> >> Signed-off-by: Eric Dong <[email protected]> >> >> Reviewed-by: Liming Gao <[email protected]> >> >> ===== >> >> >> >> Thanks, >> >> >> >> Mike >> >> >> ------------------------------------------------------------------------------ >> Want fast and easy access to all the code in your enterprise? Index and >> search up to 200,000 lines of code with a free copy of Black Duck >> Code Sight - the same software that powers the world's largest code >> search on Ohloh, the Black Duck Open Hub! Try it now. >> http://p.sf.net/sfu/bds >> _______________________________________________ >> edk2-buildtools-devel mailing list >> [email protected] >> https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel >> ------------------------------------------------------------------------------ Want fast and easy access to all the code in your enterprise? Index and search up to 200,000 lines of code with a free copy of Black Duck Code Sight - the same software that powers the world's largest code search on Ohloh, the Black Duck Open Hub! Try it now. http://p.sf.net/sfu/bds _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel
