[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/8] travis: Run bootstrap instead of autogen.sh
From: |
Daniel Kiper |
Subject: |
Re: [PATCH v2 1/8] travis: Run bootstrap instead of autogen.sh |
Date: |
Fri, 10 May 2019 13:03:16 +0200 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Thu, May 02, 2019 at 08:55:30AM +0200, Alexander Graf wrote:
> Commit 35b909062e7b3 ("gnulib: Upgrade Gnulib and switch to bootstrap tool")
> changed the build flow from running ./autogen.sh to running ./bootstrap
> but missed to update .travis.yml. Adapt it accordingly.
>
> Fixes: 35b909062e7b3 ("gnulib: Upgrade Gnulib and switch to bootstrap tool")
> Signed-off-by: Alexander Graf <address@hidden>
>
> ---
>
> v1 -> v2:
>
> - use local gnulib copy (saves ~3m git checkout time)
> ---
> .travis.yml | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index 81de20fa3..6b8f97830 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -18,6 +18,7 @@ addons:
> - python
> - qemu-system
> - unifont
> + - gnulib
>
> env:
> global:
> @@ -35,7 +36,11 @@ before_script:
> script:
> # Comments must be outside the command strings below, or the Travis parser
> # will get confused.
> - - ./autogen.sh
> +
> + # The Ubuntu gnulib does not compile with gcc8, so only use it with the
> system compiler
> + - if [ ! "$CROSS_TARGETS" ]; then rm -rf gnulib; ln -s /usr/share/gnulib;
> fi
I would not depend on it at all. Could you drop this and gnulib package
addition above?
Daniel
- Re: [PATCH v2 7/8] travis: Add ARM thumb target to tests, (continued)
[PATCH v2 8/8] travis: Test module loading from tftp as well, Alexander Graf, 2019/05/02
[PATCH v2 2/8] travis: Fix sparc64 test, Alexander Graf, 2019/05/02
[PATCH v2 4/8] travis: Fix arm coreboot test and make loop more robus, Alexander Graf, 2019/05/02
[PATCH v2 5/8] arm coreboot: Use common directory path, Alexander Graf, 2019/05/02
[PATCH v2 1/8] travis: Run bootstrap instead of autogen.sh, Alexander Graf, 2019/05/02
- Re: [PATCH v2 1/8] travis: Run bootstrap instead of autogen.sh,
Daniel Kiper <=
[PATCH v2 6/8] travis: Add smoke tests for arm and aarch64, Alexander Graf, 2019/05/02
Re: [PATCH v2 0/8] Travis fixes, Daniel Kiper, 2019/05/10