Looks good and thank you for sorting this out. Chris
On 17/8/21 1:17 am, Christian MAUDERER wrote: > Am 16.08.21 um 17:12 schrieb Joel Sherrill: >> On Mon, Aug 16, 2021 at 10:08 AM Christian MAUDERER >> <christian.maude...@embedded-brains.de> wrote: >>> >>> Hello Joel, >>> >>> Am 16.08.21 um 17:03 schrieb Joel Sherrill: >>>> On Mon, Aug 16, 2021 at 9:49 AM Christian MAUDERER >>>> <christian.maude...@embedded-brains.de> wrote: >>>>> >>>>> Hello, >>>>> >>>>> I noted that I can't build the RTEMS 5 toolchain if I have a modern LLVM >>>>> installed using the RTEMS source builder. Therefore I would like to >>>>> backport the patch 37ad446d9dce3 ([1]) to the 5 branch. Is that OK? If >>>>> someone acknowledges it, I would do the following steps without further >>>>> review (but of course with testing): >>>>> >>>>> 1. Create a ticket for the problem. >>>>> 2. In rtems-tools: Cherry pick the patch 37ad446d9dce3 to the 5 branch. >>>>> 3. In rtems-source-builder: Backport the patch 91932d0a0 [2] to the 5 >>>>> branch (or rather re-create it because it will be a different commit and >>>>> a different hash). >>>> >>>> I'm ok with this. I remember doing a patch for the opposite direction. >>>> I couldn't >>>> build on an older gcc because it assumed a different language version as >>>> default. gcc 4.8 as on CentOS 7 assumes C++03. Newer versions assume C++11 >>>> as the base. Similarly newer gcc's seem to default to C11. >>> >>> Thanks. I'll wait a bit more before adding the patches in case someone >>> else want's to object. >> >> >> Good plan. >> >>>> >>>> I posted a few weeks ago that users should really be careful to always use >>>> -std and consciously pick what they want. >>> >>> In this case, the version of the standard depends quite a lot on the >>> LLVM headers. LLVM seems to love using new features. >> >> Then this is probably not the first time we will face this. > > Definitively not. It is only a backport of a patch so I had that problem > already > on another system. I only missed to add it to the 5 branch back then. > > Best regards > > Christian > >> >> --joel >> >>> Best regards >>> >>> Christian >>> >>>> >>>> --joel >>>> >>>>> >>>>> Best regards >>>>> >>>>> Christian >>>>> >>>>> [1] >>>>> https://git.rtems.org/rtems-tools/commit/?id=37ad446d9dce3438d6d32e1caf56d3fdccdd2ad0 >>>>> >>>>> >>>>> [2] >>>>> https://git.rtems.org/rtems-source-builder/commit/?id=91932d0a0c86400366f9c75a62123b6488ff458a >>>>> >>>>> -- >>>>> -------------------------------------------- >>>>> embedded brains GmbH >>>>> Herr Christian MAUDERER >>>>> Dornierstr. 4 >>>>> 82178 Puchheim >>>>> Germany >>>>> email: christian.maude...@embedded-brains.de >>>>> phone: +49-89-18 94 741 - 18 >>>>> fax: +49-89-18 94 741 - 08 >>>>> >>>>> Registergericht: Amtsgericht München >>>>> Registernummer: HRB 157899 >>>>> Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler >>>>> Unsere Datenschutzerklärung finden Sie hier: >>>>> https://embedded-brains.de/datenschutzerklaerung/ >>>>> _______________________________________________ >>>>> devel mailing list >>>>> devel@rtems.org >>>>> http://lists.rtems.org/mailman/listinfo/devel >>> >>> -- >>> -------------------------------------------- >>> embedded brains GmbH >>> Herr Christian MAUDERER >>> Dornierstr. 4 >>> 82178 Puchheim >>> Germany >>> email: christian.maude...@embedded-brains.de >>> phone: +49-89-18 94 741 - 18 >>> fax: +49-89-18 94 741 - 08 >>> >>> Registergericht: Amtsgericht München >>> Registernummer: HRB 157899 >>> Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler >>> Unsere Datenschutzerklärung finden Sie hier: >>> https://embedded-brains.de/datenschutzerklaerung/ > _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel