Liming, I understand the benefit to get a stable BaseTools in EDK2. And it looks your level of validation is high as the BaseTools sync was announced a week ago and it has not been pushed yet.
But if when you will sync the BaseTools my firmware does not work anymore (it has been the case in the past), then I will have to find out in the last 75 patches the one that breaks my firmware. It might be my fault as it can a defect in the new BaseTools (... it also been the case in the past). I could based my development on the BaseTools project, but I will have to maintain two lists of BaseTools patchset - one for the BaseTools project and one for the EDK2 BaseTools as most developers used the EDK2 BaseTools as the reference BaseTools. Having BaseTools merged into EDK2 will ensure a regular testing by the community. In this scheme, if BaseTools introduces a regression, it would be identified and fixed more quickly. I cannot disagree when you say a defect in the BaseTools would be difficult to identify by most developers. But the way it works today is not good neither. I already know when the BaseTools will be synced, I would need to tell everyone they need to patch it because it is broken ... I am glad we can request for a BaseTools sync (even if in the last 15 months I have seen a couple of emails on the EDK2 mailing-list suggesting a sync - but no action), as soon as I will have my fixes for GenFw and GCC48 into SVN, I will request new BaseTools syncs :-) As usual the discussions about the EDK2 project (see threads on git repo, platform support, etc) will stay on the mailing-list. And we will never get an official response back from the EDK2 project responsible. I have to admit I am not sure who they are - in this list https://sourceforge.net/p/edk2/_members/, I do not recognize any name of EDK2 repository responsible. Olivier > -----Original Message----- > From: Gao, Liming [mailto:[email protected]] > Sent: 21 August 2013 09:17 > To: 'Andrew Fish'; Justen, Jordan L > Cc: [email protected]; edk2-buildtools- > [email protected]; Olivier Martin > Subject: RE: [edk2-buildtools] [edk2] Sync BaseTools Trunk (version > r2598) to EDKII main trunk > > Andrew and Martin: > I own BaseToolsSync. The sync depends on the request, no regular > plan. If you have the request, you can raise it to me. Before sync, I > need make sure BaseTools Trunk has no regression issue. > > One main point to separate BaseTools project is to make sure EDKII > BaseTools is a stable version. BaseTools is different from Shell. If > BaseTools has the issue, it may cause build failure and block > development. And, most developers are not familiar with EDKII > BaseTools. Build failure or other build issues are hard for developers > to know the root cause. > > Besides, this round sync will use version r2599. One patch is missing > in r2598. > > Thanks > Liming > -----Original Message----- > From: Andrew Fish [mailto:[email protected]] > Sent: Wednesday, August 21, 2013 6:48 AM > To: Justen, Jordan L > Cc: [email protected]; edk2-buildtools- > [email protected]; Olivier Martin > Subject: Re: [edk2-buildtools] [edk2] Sync BaseTools Trunk (version > r2598) to EDKII main trunk > > > On Aug 20, 2013, at 3:41 PM, Jordan Justen <[email protected]> > wrote: > > > On Tue, 2013-08-20 at 09:45 -0700, Andrew Fish wrote: > >> The current BaseTools process does not seem to work for any one who > >> does not have a @intel.com email address, and that seems like a > >> broken thing for an open source project. > > > > What difference does the @intel.com email address make with regards > to > > this? I've been having to use BaseTools from trunk rather than > > edk2/BaseTools for nearly a year to have GCC 4.7 support. > > > > Have you been nagging the mailing list or @intel.com? > > > Now, apparently you have to have an @intel.com email address to > > participate in code reviews for most contributions from Intel, but > > that is another matter. :) > > > > Or to know any information about the schedule roadmap or plans..... > Syncs from the BaseTools project just see to fall randomly from the sky > based on some priority not related to some targets, processors or > toolchains.... > > > Another question I have relates to BaseTools/Bin/Win32. > > Why is this not an extra download rather than part of the *source* > > tree? > > > > Well there is not a good scheme to pull what you want.... For Visual > Studio development folks generally use the frozen Python and it is much > easier to just pull the binary. > > Thanks, > > Andrew Fish > > > -Jordan > > > >> On Aug 20, 2013, at 4:24 AM, Olivier Martin <[email protected]> > wrote: > >> > >>> Does the BaseTools project really need to be out of the EDK2 > >>> project? Why do we not merge both projects? > >>> > >>> The only reason I can see is to get the Windows BaseTools Binaries > >>> aligned with the BaseTools synced version. But we also have the > case > >>> with the ShellPkg/ShellBinPkg where the ShellBinPkg are not > >>> necessary aligned with the ShellPkg sources. Nothing prevents > anyone > >>> to rebuild the Windows Binaries if needed. > >>> > >>> Adding a new toolchain to the BaseTools project does not require to > >>> rebuild the binaries. But we will have to wait the next sync to get > it. > >>> GCC4.8 has been released almost 3 months ago people will start to > >>> use it to build EDK2. But BaseTools does not support yet (I am also > >>> responsible for it > >>> - I could submit a patch by myself for such support). > >>> > >>> > >>>> -----Original Message----- > >>>> From: Jordan Justen [mailto:[email protected]] > >>>> Sent: 19 August 2013 05:48 > >>>> To: [email protected] > >>>> Subject: Re: [edk2] Sync BaseTools Trunk (version r2598) to EDKII > >>>> main trunk > >>>> > >>>> On Sun, Aug 18, 2013 at 7:57 PM, Gao, Liming > <[email protected]> > >>>> wrote: > >>>>> Martin: > >>>>> > >>>>> If so, this patch may not catch this round sync. But, you can > >>>>> still > >>>> try > >>>>> your best to commit it. > >>>> > >>>> Hopefully it won't be another 15 months until the next sync. :) > >>>> > >>>> -Jordan > >>>> > >>>>> From: Olivier Martin [mailto:[email protected]] > >>>>> Sent: Friday, August 16, 2013 10:43 PM > >>>>> > >>>>> > >>>>> To: [email protected] > >>>>> Subject: Re: [edk2] Sync BaseTools Trunk (version r2598) to EDKII > >>>> main trunk > >>>>> > >>>>> > >>>>> > >>>>> I am waiting to get the new version of my patch reviewed before > to > >>>> commit it > >>>>> into BaseTools trunk. > >>>>> > >>>>> This patch fixes the calculation of the PE/COFF header attribute > >>>>> 'BaseOfCode' in GenFw. It affects all the toolchains (and > >>>> architectures) > >>>>> that use this tool to convert ELF to PE/COFF binary (mainly the > >>>>> GCC toolchains). > >>>>> > >>>>> > >>>>> > >>>>> > >>>>> > >>>>> From: Gao, Liming [mailto:[email protected]] > >>>>> > >>>>> Sent: 16 August 2013 04:27 > >>>>> To: [email protected] > >>>>> Subject: Re: [edk2] Sync BaseTools Trunk (version r2598) to EDKII > >>>> main trunk > >>>>> > >>>>> > >>>>> > >>>>> Have you committed it into BaseTools trunk? If yes, I will sync > it > >>>> together. > >>>>> If no, I want to know what impact for this patch. Does it impact > >>>>> all > >>>> GCC > >>>>> build or only ARM GCC build? > >>>>> > >>>>> > >>>>> > >>>>> From: Olivier Martin [mailto:[email protected]] > >>>>> Sent: Thursday, August 15, 2013 5:14 PM > >>>>> To: [email protected] > >>>>> Subject: Re: [edk2] Sync BaseTools Trunk (version r2598) to EDKII > >>>> main trunk > >>>>> > >>>>> > >>>>> > >>>>> I sent a patch on June 11th that fixes the calculation of the > >>>>> PE/COFF > >>>> header > >>>>> attribute 'BaseOfCode'. And a second version on July 16th taking > >>>>> in > >>>> account > >>>>> the comments after the review. > >>>>> > >>>>> Do you think my fix could be integrated to this BaseTools sync? > >>>>> > >>>>> > >>>>> > >>>>> Thanks, > >>>>> > >>>>> Olivier > >>>>> > >>>>> > >>>>> > >>>>> > >>>>> > >>>>> From: Gao, Liming [mailto:[email protected]] > >>>>> Sent: 15 August 2013 03:36 > >>>>> To: [email protected] > >>>>> Subject: [edk2] Sync BaseTools Trunk (version r2598) to EDKII > main > >>>> trunk > >>>>> > >>>>> > >>>>> > >>>>> Hi, all > >>>>> > >>>>> I will sync BaseTool trunk (version r2598) into EDKII BaseTools. > >>>>> The > >>>> main > >>>>> changes include: > >>>>> > >>>>> 1. Python version is updated from 2.5.4 to > 2.7.3. > >>>> The > >>>>> binary image are freezed based on Python2.7.3 and cx_Freeze > 4.2.3. > >>>>> > >>>>> 2. Add GCC47 and VS2012 tool chain. > >>>>> > >>>>> 3. Enable EFI_IFR_DEFAULT2 opcode in vfrcompile > to > >>>> save > >>>>> image size. > >>>>> > >>>>> 4. Add Aarch64 ARM architecture > >>>>> > >>>>> > >>>>> > >>>>> BaseTool Trunk: > >>>>> > >>>>> https://svn.code.sf.net/p/edk2-buildtools/code/trunk/BaseTools > >>>>> > >>>>> EDKII BaseTools: > >>>>> > >>>>> https://svn.code.sf.net/p/edk2/code/trunk/edk2/BaseTools > >>>>> > >>>>> > >>>>> > >>>>> If you have any comments, please let me know. > >>>>> > >>>>> > >>>>> > >>>>> Thanks > >>>>> > >>>>> Liming > >>>>> > >>>>> > >>>>> ----------------------------------------------------------------- > - > >>>>> --- > >>>> --------- > >>>>> Get 100% visibility into Java/.NET code with AppDynamics Lite! > >>>>> It's a free troubleshooting tool designed for production. > >>>>> Get down to code-level detail for bottlenecks, with <2% overhead. > >>>>> Download for free and get started troubleshooting in minutes. > >>>>> > >>>> > http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ost > >>>> g.cl > >>>> ktrk > >>>>> _______________________________________________ > >>>>> edk2-devel mailing list > >>>>> [email protected] > >>>>> https://lists.sourceforge.net/lists/listinfo/edk2-devel > >>>>> > >>>> > >>>> ------------------------------------------------------------------ > - > >>>> ---- > >>>> ------- > >>>> Get 100% visibility into Java/.NET code with AppDynamics Lite! > >>>> It's a free troubleshooting tool designed for production. > >>>> Get down to code-level detail for bottlenecks, with <2% overhead. > >>>> Download for free and get started troubleshooting in minutes. > >>>> > http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ost > >>>> g.cl > >>>> ktrk > >>>> _______________________________________________ > >>>> edk2-devel mailing list > >>>> [email protected] > >>>> https://lists.sourceforge.net/lists/listinfo/edk2-devel > >>> > >>> > >>> > >>> > >>> > >>> ------------------------------------------------------------------- > - > >>> ---------- Introducing Performance Central, a new site from > >>> SourceForge and AppDynamics. Performance Central is your source for > >>> news, insights, analysis and resources for efficient Application > >>> Performance Management. > >>> Visit us today! > >>> > http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg > >>> .clktrk _______________________________________________ > >>> edk2-buildtools-devel mailing list > >>> [email protected] > >>> https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel > >> > >> > >> -------------------------------------------------------------------- > - > >> --------- Introducing Performance Central, a new site from > >> SourceForge and AppDynamics. Performance Central is your source for > >> news, insights, analysis and resources for efficient Application > >> Performance Management. > >> Visit us today! > >> > http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg. > >> clktrk _______________________________________________ > >> edk2-buildtools-devel mailing list > >> [email protected] > >> https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel > > > > > > > ----------------------------------------------------------------------- > ------- > Introducing Performance Central, a new site from SourceForge and > AppDynamics. Performance Central is your source for news, insights, > analysis and resources for efficient Application Performance > Management. > Visit us today! > http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg.cl > ktrk > _______________________________________________ > edk2-buildtools-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel ------------------------------------------------------------------------------ Introducing Performance Central, a new site from SourceForge and AppDynamics. Performance Central is your source for news, insights, analysis and resources for efficient Application Performance Management. Visit us today! http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg.clktrk _______________________________________________ edk2-buildtools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel
