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...
> -----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-devel > > >> > > > > > > > > --------------------------------------------------------------------- > -- > > ------- > > 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
