[Buildroot] [PATCH v3 01/10] package/rustc: install cargo config as toolchain post-install hook

[David PIERRET]

Hi,
Working on a new package using rust, this patch suite have resolved my
issue with rust version successfully.

Regards,
David

On Fri, Feb 7, 2020 at 7:08 PM Arnout Vandecappelle (Essensium/Mind)
<arnout at mind.be> wrote:
>
> The cargo config file is much like the cmake and meson configuration
> files, something that tells the cargo build system how the
> cross-toolchain is configured. It can be used not only with our own
> host-cargo package, but also with a pre-installed cargo.
>
> Therefore, install the cargo config file as a toolchain post-install
> hook, just like we do for cmake and meson. However, we don't have a
> pkg-cargo infrastructure (yet) so we can't put it there. The cargo
> package itself will be removed soon, so that's not appropriate either.
> Therefore, put it in the rustc package - any cargo package will need
> rustc.
>
> Remove the config handling from the cargo package.
>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
Tested-by: David Pierret <david.pierret at smile.fr>
> ---
>  package/cargo/cargo.mk                             |  6 ------
>  package/{cargo/config.in => rustc/cargo-config.in} |  0
>  package/rustc/rustc.mk                             | 12 ++++++++++++
>  3 files changed, 12 insertions(+), 6 deletions(-)
>  rename package/{cargo/config.in => rustc/cargo-config.in} (100%)
>
> diff --git a/package/cargo/cargo.mk b/package/cargo/cargo.mk
> index a387281b4c..3f6646d027 100644
> --- a/package/cargo/cargo.mk
> +++ b/package/cargo/cargo.mk
> @@ -80,12 +80,6 @@ endef
>
>  define HOST_CARGO_INSTALL_CMDS
>         $(INSTALL) -D -m 0755 $(@D)/target/release/cargo $(HOST_DIR)/bin/cargo
> -       $(INSTALL) -D package/cargo/config.in \
> -               $(HOST_DIR)/share/cargo/config
> -       $(SED) 's/@RUSTC_TARGET_NAME@/$(RUSTC_TARGET_NAME)/' \
> -               $(HOST_DIR)/share/cargo/config
> -       $(SED) 's/@CROSS_PREFIX@/$(notdir $(TARGET_CROSS))/' \
> -               $(HOST_DIR)/share/cargo/config
>  endef
>
>  $(eval $(host-generic-package))
> diff --git a/package/cargo/config.in b/package/rustc/cargo-config.in
> similarity index 100%
> rename from package/cargo/config.in
> rename to package/rustc/cargo-config.in
> diff --git a/package/rustc/rustc.mk b/package/rustc/rustc.mk
> index 6eea9b4fc5..35ffc36106 100644
> --- a/package/rustc/rustc.mk
> +++ b/package/rustc/rustc.mk
> @@ -20,3 +20,15 @@ endif
>  RUSTC_HOST_NAME = $(RUSTC_HOST_ARCH)-unknown-linux-gnu
>
>  $(eval $(host-virtual-package))
> +
> +ifeq ($(BR2_PACKAGE_HOST_RUSTC_TARGET_ARCH_SUPPORTS),y)
> +define RUSTC_INSTALL_CARGO_CONFIG
> +       mkdir -p $(HOST_DIR)/share/cargo
> +       sed -e 's/@RUSTC_TARGET_NAME@/$(RUSTC_TARGET_NAME)/' \
> +               -e 's/@CROSS_PREFIX@/$(notdir $(TARGET_CROSS))/' \
> +               package/rustc/cargo-config.in \
> +               > $(HOST_DIR)/share/cargo/config
> +endef
> +# check-package disable TypoInPackageVariable - TOOLCHAIN intended
> +TOOLCHAIN_POST_INSTALL_STAGING_HOOKS += RUSTC_INSTALL_CARGO_CONFIG
> +endif
> --
> 2.24.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot