Re: [PATCH] RISC-V: Canonicalize --with-arch

[Kito Cheng <kito.cheng@gmail.com>]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98152

Andreas Schwab has created a bug entry for this issue,
using awk or shell should be fine to get the same functionality, but
it might take some time,
so I plan to add some checking to detect python, python2 or python3,
and skip this step if none of above is found.
The arch string canonicalize is not a must, it's a kind of improvement,
trying to reduce the build # of multi-lib, so I think skip that should
not be a problem if python is not available.

On Fri, Dec 4, 2020 at 10:41 PM Matthias Klose <doko@ubuntu.com> wrote:
>
> On 12/4/20 2:38 PM, Matthias Klose wrote:
> > On 12/4/20 9:07 AM, Kito Cheng via Gcc-patches wrote:
> >> Committed, thanks :)
> >>
> >> On Thu, Dec 3, 2020 at 8:51 AM Jim Wilson <jimw@sifive.com> wrote:
> >>>
> >>> On Tue, Dec 1, 2020 at 12:13 AM Kito Cheng <kito.cheng@sifive.com> wrote:
> >>>>
> >>>>  - We would like to canonicalize the arch string for --with-arch for
> >>>>    easier handling multilib, so split canonicalization part to a stand
> >>>>    along script to shared the logic.
> >>>>
> >>>> gcc/ChangeLog:
> >>>>
> >>>>         * config/riscv/multilib-generator (arch_canonicalize): Move
> >>>>         code to arch-canonicalize, and call that script to canonicalize arch
> >>>>         string.
> >>>>         (canonical_order): Move code to arch-canonicalize.
> >>>>         (LONG_EXT_PREFIXES): Ditto.
> >>>>         (IMPLIED_EXT): Ditto.
> >>>>         * config/riscv/arch-canonicalize: New.
> >>>>         * config.gcc (riscv*-*-*): Canonicalize --with-arch.
> >>>
> >>>
> >>> Looks OK to me.
> >
> > that breaks the bootstrap if python is not available. The python command might
> > not be available, so please check for python3, python, or python2.
>
> same for config/riscv/arch-canonicalize