Re: [PATCHv2] build.sh: yet another fix for the installation of build deps

[Henning Schild <henning.schild@siemens.com>]

Changes in v2, install "equivs" as well, otherwise the tool will not
run.

Henning

Am Thu, 4 Jan 2018 10:44:50 +0100
schrieb Henning Schild <henning.schild@siemens.com>:

> The perl code guessing the build-deps of packages can not handle all
> the possible syntax that could be found in Build-Depends. And it
> ignores Build-Depends-Indep. So there are several issues in it.
> 
> This patch uses an official debian-tool "mk-build-deb" to install the
> build deps. The tool generates a package which which will Depends: on
> the Build-Depends:. Architecture operators, -Indep etc. are handled
> correctly.
> 
> Signed-off-by: Henning Schild <henning.schild@siemens.com>
> ---
>  meta/recipes-devtools/buildchroot/buildchroot.bb | 4 +++-
>  meta/recipes-devtools/buildchroot/files/build.sh | 9 ++-------
>  2 files changed, 5 insertions(+), 8 deletions(-)
> 
> diff --git a/meta/recipes-devtools/buildchroot/buildchroot.bb
> b/meta/recipes-devtools/buildchroot/buildchroot.bb index
> 59ad0e0..51f9d5d 100644 ---
> a/meta/recipes-devtools/buildchroot/buildchroot.bb +++
> b/meta/recipes-devtools/buildchroot/buildchroot.bb @@ -25,7 +25,9 @@
> BUILDCHROOT_PREINSTALL ?= "gcc \ locales \
>                             docbook-to-man \
>                             apt \
> -                           automake"
> +                           automake \
> +                           devscripts \
> +                           equivs"
>  
>  WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}"
>  
> diff --git a/meta/recipes-devtools/buildchroot/files/build.sh
> b/meta/recipes-devtools/buildchroot/files/build.sh index
> 5e310ad..975252e 100644 ---
> a/meta/recipes-devtools/buildchroot/files/build.sh +++
> b/meta/recipes-devtools/buildchroot/files/build.sh @@ -6,13 +6,8 @@
>  # Go to build directory
>  cd $1
>  
> -# Get list of dependencies manually. The package is not in apt, so
> no apt-get -# build-dep. dpkg-checkbuilddeps output contains version
> information and isn't -# directly suitable for apt-get install.
> -DEPS=`perl -ne 'next if /^#/; $p=(s/^Build-Depends:\s*/ / or (/^ /
> and $p)); s/,|\n|\([^)]+\)|\[[^]]+\]//mg; print if $p' <
> debian/control` - -# Install deps
> -apt-get install -y $DEPS
> +# Install all build deps
> +mk-build-deps -i -r debian/control
>  
>  # If autotools files have been created, update their timestamp to
>  # prevent them from being regenerated