Re: Building TCC with the upcoming Mes 0.26

[Andrius Štikonas]

Yes, that should be restricted to riscv64.

But that should also fail on mes 0.25 then...

Andrius

2023 m. lapkričio 23 d., ketvirtadienis 19:36:17 GMT Timothy Sample rašė:
> Ekaitz Zarraga <ekaitz@elenq.tech> writes:
> 
> > On 11/23/23 08:48, Janneke Nieuwenhuizen wrote:
> >> Timothy Sample writes:
> >>> The first was that I had to revert the ‘bootstrap.sh’ changes from
> >>> commit 0d3c4ddc4bcb57db690ee8ebab106fa6fda75823.  Not sure what that’s
> >>> about.
> >> The commit message is prettry clear?  It's for tcc riscv64 (it only
> >> knows a simplified assembly).  Do you know how this broke your build,
> >> I believe Andrius even built tcc for x86?
> 
> It’s line 32 of “build-source-lib.sh”.  It tries to copy from:
> 
>     ${srcdest}lib/$mes_kernel/$mes_cpu-mes-$compiler
> 
> Which is “../lib/linux/x86-mes-tcc”, and which doesn’t exist.
> 
> > This makes some sense, actually.
> >
> > Andrius built it but not using these scripts but their own from
> > live-bootstrap.
> >
> > So this is a very valuable feedback.
> > Maybe we need to hide this part of the commit under an `if`?
> >
> > +(
> > +    mkdir -p tcc-lib
> > +    cp config.sh tcc-lib
> > +    cd tcc-lib
> > +    compiler=tcc
> > +    if test -z "$srcdest"; then
> > +        srcdest=../
> > +        srcdir=../
> > +    fi
> > +    ${SHELL} ${srcdest}build-aux/build-source-lib.sh
> > +)
> 
> I don’t pretend to understand the build system super well, but it would
> fix it to hide it behind:
> 
>     if test -d ${srcdest}lib/$mes_kernel/$mes_cpu-mes-tcc
> 
> Here’s a patch for that.  I tested it and was able to do the build with
> it.  I did not test that it works for the RISC-V build (I wouldn’t even
> know how!).
> 
> 
> -- Tim
> 
>