Hi, I find CYGGCC is also listed in tools_def.txt. Seemly, no one test it. I will verify it and let you result.
Oliver: Could you wait me one or two day to verify CYGGCC, then decide to commit this patch? Thanks Liming -----Original Message----- From: Jordan Justen [mailto:[email protected]] Sent: Thursday, February 13, 2014 3:04 AM To: Olivier Martin Cc: [email protected]; Kirkendall, Garrett Subject: Re: [edk2-buildtools] [2nd Request] [PATCH] BaseTools/Conf/build_rule.template: modify GCC link commands to be more OS agnostic On Wed, Feb 12, 2014 at 7:41 AM, Olivier Martin <[email protected]> wrote: > If no one has any objection by the end of the week, I will push > Garrett's change into BaseTools-svn. > Garrett sent the patch last October... For GCC47 Tested-by: Jordan Justen <[email protected]> I checked the older OS X machines in our build pool. (Which was about the only thing I could think of that might have an issue with this.) It seems like the default ld installed by xcode (at that time) did not support --start-group. But, for building EDK II, we use a separate 2.20 binutils on those machines. It does seem to support --start-group. So, I don't expect this change will break the builds on those machines. -Jordan >> -----Original Message----- >> From: Olivier Martin [mailto:[email protected]] >> Sent: 24 January 2014 18:03 >> To: 'Kirkendall, Garrett'; >> [email protected] >> Subject: RE: [edk2-buildtools] [2nd Request] [PATCH] >> BaseTools/Conf/build_rule.template: modify GCC link commands to be >> more OS agnostic >> >> (hidden request) I confirm this patch does not break the ARM GCC >> toolchain. >> >> Tested-by: Olivier Martin <[email protected]> >> >> >> > -----Original Message----- >> > From: Kirkendall, Garrett [mailto:[email protected]] >> > Sent: 10 January 2014 21:44 >> > To: [email protected] >> > Subject: Re: [edk2-buildtools] [2nd Request] [PATCH] >> > BaseTools/Conf/build_rule.template: modify GCC link commands to be >> more >> > OS agnostic >> > >> > I contacted Andrew Fish with Apple and he looked at my patch. >> > XCODE already has its own <Command.XCODE> for linking so it would >> > not be affected by this change. The only other problem area that >> > Jorden >> could >> > think of was maybe the UNIXGCC, but it requires GCC 4.3 as >> > designated in the tools_def.txt file. The earliest 4.3.1 GCC was >> > released in 2008, so I have to assume it would contain ld with a >> > version later >> than >> > 2.13.2.1 (2002) which I found supports the --start-group and --end- >> > group parameters. >> > >> > Does anyone know of any other reason not to accept this patch? >> > >> > GARRETT KIRKENDALL >> > SMTS Firmware Engineer | AMD Technology & Engineering >> > 7171 Southwest Parkway, Austin, TX 78735 USA >> > facebook | amd.com >> > >> > >> > -----Original Message----- >> > From: Kirkendall, Garrett >> > Sent: Tuesday, November 12, 2013 2:06 PM >> > To: Jordan Justen >> > Cc: [email protected] >> > Subject: Re: [edk2-buildtools] [2nd Request] [PATCH] >> > BaseTools/Conf/build_rule.template: modify GCC link commands to be >> more >> > OS agnostic >> > >> > Justen, thank you very much for doing some testing with this change. >> > Hopefully we'll get some input on UNIXGCC and OS X. >> > >> > GARRETT KIRKENDALL >> > SMTS Firmware Engineer | AMD Technology & Engineering >> > 7171 Southwest Parkway, Austin, TX 78735 USA >> > facebook | amd.com >> > >> > -----Original Message----- >> > From: Jordan Justen [mailto:[email protected]] >> > Sent: Saturday, November 09, 2013 4:52 PM >> > To: Kirkendall, Garrett >> > Cc: Gao, Liming; [email protected] >> > Subject: Re: [edk2-buildtools] [2nd Request] [PATCH] >> > BaseTools/Conf/build_rule.template: modify GCC link commands to be >> more >> > OS agnostic >> > >> > On Fri, Nov 8, 2013 at 9:50 PM, Jordan Justen <[email protected]> >> > wrote: >> > > On Fri, Nov 8, 2013 at 6:06 AM, Kirkendall, Garrett >> > > <[email protected]> wrote: >> > >> >> > >> I verified with ARMGCC and booted. I did a build of >> > >> MdeModulePkg/MdeModulePkg.dsc with GCC47 and the ld commands >> worked >> > >> without a problem and I checked that HelloWorld.efi was produced. >> > >> > I verified OVMF IA32 & X64 built with GCC47 is functional with this >> > change. Not a big surprise. >> > >> > >> I looked at ld --help for gnu ld version 2.13.2.1 (2002) and >> > >> it claims to support --start-group and --end-group, so I assume >> > >> it >> will >> > >> work properly in most any ld more recent than that. >> > > >> > > I can't remember where, but it seemed like this was a problem for >> > > binutils somewhere. It would have been 3~4 years ago. >> > > >> > > It seems like it might have been mingw (UNIXGCC) on windows or OSX. >> > > Hmm, I'm thinking OSX is the more likely suspect. >> > >> > I don't think this will be an issue for GCC44-47. >> > >> > A bit of searching seemed to turn up some references to 'non-GNU' >> > ld not supporting --start-group. Not sure how this might impact edk2. >> > >> > Can someone check UNIXGCC on OS X? I don't think we deprecated >> UNIXGCC, >> > even though GCC44+ is a better option on Linux, and I think XCLANG >> > is >> a >> > better option on OS X. >> > >> > -Jordan >> > >> > >> From: Gao, Liming [mailto:[email protected]] >> > >> Sent: Friday, November 08, 2013 2:03 AM >> > >> To: Kirkendall, Garrett; edk2-buildtools- >> [email protected] >> > >> Subject: RE: [edk2-buildtools] [2nd Request] [PATCH] >> > >> BaseTools/Conf/build_rule.template: modify GCC link commands to >> > >> be more OS agnostic >> > >> >> > >> >> > >> >> > >> Hi: >> > >> >> > >> This change is good to me. Have you verified it in GCC and >> ARMGCC? >> > >> >> > >> >> > >> >> > >> Thanks >> > >> >> > >> Liming >> > >> >> > >> From: Kirkendall, Garrett [mailto:[email protected]] >> > >> Sent: Thursday, November 7, 2013 10:10 PM >> > >> To: [email protected] >> > >> Subject: [edk2-buildtools] [2nd Request] [PATCH] >> > >> BaseTools/Conf/build_rule.template: modify GCC link commands to >> > >> be more OS agnostic >> > >> >> > >> >> > >> >> > >> Does anyone see a potential problem with this change? No one has >> > responded with any input or accepted the change. >> > >> >> > >> >> > >> >> > >> Garrett Kirkendall >> > >> SMTS Firmware Engineer | AMD Technology & Engineering >> > >> 7171 Southwest Parkway, Austin, TX 78735 USA O +(1) 512-602-3255 >> > >> facebook | amd.com >> > >> >> > >> >> > >> >> > >> From: Kirkendall, Garrett >> > >> Sent: Wednesday, October 16, 2013 11:28 AM >> > >> To: [email protected] >> > >> Subject: [edk2-buildtools] [PATCH] >> > >> BaseTools/Conf/build_rule.template: modify GCC link commands to >> > >> be more OS agnostic >> > >> >> > >> >> > >> >> > >> Dear BaseTools maintainers, >> > >> >> > >> >> > >> >> > >> Please consider the attached patch that allows GCC ld to run >> > >> under >> > Windows while using the archive command-line parameters. The short >> > version options are replaced with the long version options. When >> > running ld directly under Windows it doesn't require the "(" or ")" >> to >> > be escaped. Changing to the long version should make the commands >> safe >> > for Linux and Windows. >> > >> >> > >> >> > >> >> > >> Garrett Kirkendall >> > >> SMTS Firmware Engineer | AMD Technology & Engineering >> > >> 7171 Southwest Parkway, Austin, TX 78735 USA >> > >> facebook | amd.com >> > >> >> > >> >> > >> >> > >> >> > >> ---------------------------------------------------------------- >> > >> -- >> -- >> > - >> > >> --------- November Webinars for C, C++, Fortran Developers >> > Accelerate >> > >> application performance with scalable programming models. >> > >> Explore techniques for threading, error checking, porting, and >> > >> tuning. Get the most from the latest Intel processors and >> > >> coprocessors. See abstracts and register >> > >> >> > http://pubads.g.doubleclick.net/gampad/clk?id=60136231&iu=/4140/ostg. >> > >> clktrk _______________________________________________ >> > >> edk2-buildtools-devel mailing list >> > >> [email protected] >> > >> https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-dev >> > >> el >> > >> >> > >> > >> > >> > ------------------------------------------------------------------- >> > -- >> -- >> > ------- >> > DreamFactory - Open Source REST & JSON Services for HTML5 & Native >> Apps >> > OAuth, Users, Roles, SQL, NoSQL, BLOB Storage and External API >> > Access Free app hosting. Or install the open source package on any >> > LAMP server. >> > Sign up and see examples for AngularJS, jQuery, Sencha Touch and >> > Native! >> > >> http://pubads.g.doubleclick.net/gampad/clk?id=63469471&iu=/4140/ostg. >> cl >> > ktrk >> > _______________________________________________ >> > edk2-buildtools-devel mailing list >> > [email protected] >> > https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel > > > > ---------------------------------------------------------------------- > -------- > Android apps run on BlackBerry 10 > Introducing the new BlackBerry 10.2.1 Runtime for Android apps. > Now with support for Jelly Bean, Bluetooth, Mapview and more. > Get your Android app in front of a whole new audience. Start now. > http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg. > clktrk _______________________________________________ > edk2-buildtools-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel ------------------------------------------------------------------------------ Android apps run on BlackBerry 10 Introducing the new BlackBerry 10.2.1 Runtime for Android apps. Now with support for Jelly Bean, Bluetooth, Mapview and more. Get your Android app in front of a whole new audience. Start now. http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk _______________________________________________ edk2-buildtools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel ------------------------------------------------------------------------------ Android apps run on BlackBerry 10 Introducing the new BlackBerry 10.2.1 Runtime for Android apps. Now with support for Jelly Bean, Bluetooth, Mapview and more. Get your Android app in front of a whole new audience. Start now. http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk _______________________________________________ edk2-buildtools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel
