[Buildroot] [PATCH 05/13] packages: use new $($PKG)_DL_DIR) variable
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Sat Jul 22 21:39:36 UTC 2017
Hello,
On Tue, 4 Jul 2017 18:22:03 +0200, Maxime Hadjinlian wrote:
> Instead of DL_DIR, the package should now use $(PKG)_DL_DIR to ease the
> transition into a new directory structure for DL_DIR.
>
> This commit has been generated with the following scripts:
>
> for i in package/*/*.mk; do
> if ! grep -q "\$(DL_DIR)" ${i}; then
> continue
> fi
> pkg_name="$(basename $(dirname ${i}))"
> raw_pkg_name=$(echo ${pkg_name} | tr [a-z] [A-Z] | tr '-' '_')
> pkg_dl_dir="${raw_pkg_name}_DL_DIR"
> sed -i "s/\$(DL_DIR)/\$($pkg_dl_dir)/" ${i}
> done
>
> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian at gmail.com>
I wanted to apply this one, but I believe there is a problem, see below:
> diff --git a/package/gcc/gcc.mk b/package/gcc/gcc.mk
> index b52f9456b3..64fcfc70e7 100644
> --- a/package/gcc/gcc.mk
> +++ b/package/gcc/gcc.mk
> @@ -289,7 +289,7 @@ HOST_GCC_COMMON_MAKE_OPTS = \
> gcc_cv_libc_provides_ssp=$(if $(BR2_TOOLCHAIN_HAS_SSP),yes,no)
>
> ifeq ($(BR2_CCACHE),y)
> -HOST_GCC_COMMON_CCACHE_HASH_FILES += $(DL_DIR)/$(GCC_SOURCE)
> +HOST_GCC_COMMON_CCACHE_HASH_FILES += $(GCC_DL_DIR)/$(GCC_SOURCE)
I'm not sure this one works. Indeed, there is no package named just
"gcc". There are two packages named gcc-initial and gcc-final, but no
package named "gcc", so GCC_DL_DIR doesn't exist. Only
GCC_INITIAL_DL_DIR and GCC_FINAL_DL_DIR exist.
Am I missing something here ?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
More information about the buildroot
mailing list