[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] Add travis-ci config file
From: |
Daniel Kiper |
Subject: |
Re: [PATCH v2] Add travis-ci config file |
Date: |
Mon, 18 Feb 2019 20:08:45 +0100 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Mon, Feb 18, 2019 at 07:57:35PM +0100, Daniel Kiper wrote:
> On Thu, Feb 14, 2019 at 02:12:28PM +0100, Alexander Graf wrote:
[...]
> > + - mkdir /tmp/cross
> > + # results in binaries like
> > /tmp/cross/gcc-8.1.0-nolibc/ia64-linux/bin/ia64-linux-gcc
> > + - for i in $CROSS_TARGETS; do
> > + ( cd /tmp/cross; wget -O -
> > https://mirrors.edge.kernel.org/pub/tools/crosstool/files/bin/x86_64/8.1.0/x86_64-gcc-8.1.0-nolibc-$i.tar.xz
> > | xzcat | tar x );
>
> Should not we add e.g. -t 3 to wget command line?
>
> s/mirrors.edge.kernel.org/mirrors.kernel.org/?
> s/xzcat | tar x/tar xJ/
>
> And should not we verify signatures of
> x86_64-gcc-8.1.0-nolibc-$i.tar.xz?
>
> And why we need "(...)", brackets here?
https://mirrors.edge.kernel.org/pub/tools/crosstool/ says:
These compilers are only functional for kernel builds, they cannot be
used to build userspace code. For complete toolchains, go to
https://toolchains.bootlin.com/
We build userspace code. So, maybe this is not the best fit for us.
Daniel