diff mbox

[1/2] e2fsprogs: propagate LDCONFIG=true

Message ID 20161118152427.16515-1-maxime.hadjinlian@gmail.com
State Accepted
Headers show

Commit Message

Maxime Hadjinlian Nov. 18, 2016, 3:24 p.m. UTC
We correctly set LDCONFIG=true for MAKE_OPTS, but not for the other
steps which don't inherit it, to avoid permissions error displayed
during the build, we need to set them up.

Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
---
 package/e2fsprogs/e2fsprogs.mk | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Thomas Petazzoni Dec. 11, 2016, 3:02 p.m. UTC | #1
Hello,

On Fri, 18 Nov 2016 16:24:26 +0100, Maxime Hadjinlian wrote:
> We correctly set LDCONFIG=true for MAKE_OPTS, but not for the other
> steps which don't inherit it, to avoid permissions error displayed
> during the build, we need to set them up.
> 
> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
> ---
>  package/e2fsprogs/e2fsprogs.mk | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
diff mbox

Patch

diff --git a/package/e2fsprogs/e2fsprogs.mk b/package/e2fsprogs/e2fsprogs.mk
index 022ba52..b01336a 100644
--- a/package/e2fsprogs/e2fsprogs.mk
+++ b/package/e2fsprogs/e2fsprogs.mk
@@ -57,8 +57,9 @@  ifeq ($(BR2_NEEDS_GETTEXT_IF_LOCALE),y)
 E2FSPROGS_CONF_ENV += LIBS=-lintl
 endif
 
-E2FSPROGS_MAKE_OPTS = \
-	LDCONFIG=true
+E2FSPROGS_MAKE_OPTS = LDCONFIG=true
+E2FSPROGS_INSTALL_STAGING_OPTS = LDCONFIG=true
+E2FSPROGS_INSTALL_TARGET_OPTS = LDCONFIG=true
 
 define HOST_E2FSPROGS_INSTALL_CMDS
 	$(HOST_MAKE_ENV) $(MAKE) -C $(@D) install install-libs