[Buildroot] [PATCH 03/15] package/pkg-generic.mk: Remove Info documents dir entry
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Mon Jun 21 20:51:20 UTC 2021
On Mon, 21 Jun 2021 16:11:18 +0200
Herve Codina <herve.codina at bootlin.com> wrote:
> Some packages (autotools for instance) install documentation
> files using install-info. This program adds an entry in
> the Info directory file (share/info/dir) and this causes
> TARGET_DIR and/or HOST_DIR overwrite.
>
> In order to avoid this overwrite this patch removes the Info
> directory file right after any installation.
>
> Signed-off-by: Herve Codina <herve.codina at bootlin.com>
> ---
> package/pkg-generic.mk | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
> index bb9ff4150a..2499c94746 100644
> --- a/package/pkg-generic.mk
> +++ b/package/pkg-generic.mk
> @@ -280,6 +280,7 @@ $(BUILD_DIR)/%/.stamp_host_installed:
> $(foreach hook,$($(PKG)_PRE_INSTALL_HOOKS),$(call $(hook))$(sep))
> +$($(PKG)_INSTALL_CMDS)
> $(foreach hook,$($(PKG)_POST_INSTALL_HOOKS),$(call $(hook))$(sep))
> + $(Q) rm -f $(HOST_DIR)/share/info/dir
No space between $(Q) and rm. This should perhaps use $(RM) in fact.
However, I'm not a huge fan of having this right in the middle of the
infrastructure. It feels like a small detail that gets handled in the
middle of super generic infrastructure code.
The issue is that I don't really have a good alternative proposal :-/
Instead of removing that file, ignore it in the overwrite detection,
perhaps?
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list