From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH v2 0/4] Add GCC cross compiler for arm-none-eabi. Date: Tue, 27 Sep 2016 11:03:01 +0200 Message-ID: <87d1jp90l6.fsf@gnu.org> References: <20160918063513.11027-1-rekado@elephly.net> <20160919202340.4122-1-theodoros.for@openmailbox.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:37248) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1booIB-0007Mj-H3 for guix-devel@gnu.org; Tue, 27 Sep 2016 05:03:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1booI5-000379-O8 for guix-devel@gnu.org; Tue, 27 Sep 2016 05:03:10 -0400 In-Reply-To: <20160919202340.4122-1-theodoros.for@openmailbox.org> (Theodoros Foradis's message of "Mon, 19 Sep 2016 23:23:36 +0300") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Theodoros Foradis Cc: guix-devel@gnu.org Hi Theodoros, Theodoros Foradis skribis: > The original patch series was working correctly and producing working bin= aries. Some > flags (that I had been using with 6.2.0) are missing from that version of= GCC 4.9, > so I added 6.2.0 as an extra option. I have tested it to produce working = binaries. > > Here are some modifications to Ricardo's patches for the arm-none-eabi > bare metal cross compiler. The following changes have been made: > > - I have modified xbinutils to use binutils 2.25.1 from cross-base, as it= compiles > correctly with it. The version from the svn commit that was used by Ricar= do is compiling > correct binaries as well. Thus, if it is deemed appropriate, the source f= or xbinutils can > be swapped for the previous one, with (seemingly) no difference. > > - The xgcc of the original, was failing to find the headers that newlib p= rovided. > I have set the native-cross-paths as a workaround. Not sure if there is a= better > alternative, or if the failure was my mistake. > > - A package for cross GCC 6.2.0 is added, with appropriate patches for mu= ltilib > support. > > - Newlib-arm-none-eabi and newlib-nano-arm-none-eabi have been changed to > procedures, taking an xgcc as argument, so as to facilitate building with > either version of gcc. > > - An arm-none-eabi-toolchain procedure is declared, to create toolchain p= ackages > for both gcc and newlib version. The four toolchain variables follow. Not= sure > if it's a mistake to include "nano" in the toolchain version. This all sounds reasonable to me. Ricardo was interested in using this toolchain for one specific purpose, so maybe we=E2=80=99ll want to check th= at it also works here. Ricardo: could you comment? FWIW I had commented on Ricardo=E2=80=99s patch series here: https://lists.gnu.org/archive/html/guix-devel/2016-09/msg01379.html > This is the first patch that I send in guix-devel, so please bear with me. Welcome! Sorry that it already took a bit long, but hopefully we=E2=80=99ll converge real soon! :-) Thanks, Ludo=E2=80=99.