No problem, I can wait a couple of days. > -----Original Message----- > From: Gao, Liming [mailto:[email protected]] > Sent: 14 February 2014 01:33 > To: Jordan Justen; 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 > > 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.c > lktrk > _______________________________________________ > edk2-buildtools-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel
-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2557590 ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2548782 ------------------------------------------------------------------------------ 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
