[Buildroot] [PATCH v2 5/5] package: linux-fusion: use install instead of cp
Guido Martínez
guido at vanguardiasur.com.ar
Fri Nov 7 13:16:29 UTC 2014
Hi Danomi
On Fri, Nov 07, 2014 at 07:44:50AM -0500, Danomi Manchego wrote:
> Guido
>
> On Thu, Nov 6, 2014 at 10:50 PM, Guido Martínez
> <guido at vanguardiasur.com.ar> wrote:
> > in order to not depend on the previous permissions of the file
> >
> > Signed-off-by: Guido Martínez <guido at vanguardiasur.com.ar>
> > ---
> > package/linux-fusion/linux-fusion.mk | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/package/linux-fusion/linux-fusion.mk b/package/linux-fusion/linux-fusion.mk
> > index c5e7976..81ecd8a 100644
> > --- a/package/linux-fusion/linux-fusion.mk
> > +++ b/package/linux-fusion/linux-fusion.mk
> > @@ -37,7 +37,7 @@ define LINUX_FUSION_INSTALL_TARGET_CMDS
> > INSTALL_MOD_PATH=$(TARGET_DIR) \
> > -C $(@D) install
> > mkdir -p $(LINUX_FUSION_ETC_DIR)
> > - cp -dpf package/linux-fusion/40-fusion.rules $(LINUX_FUSION_ETC_DIR)
> > + install -m 644 package/linux-fusion/40-fusion.rules $(LINUX_FUSION_ETC_DIR)
>
>
> How about also dropping the mkdir and doing:
>
> install -D -m 644 package/linux-fusion/40-fusion.rules
> $(LINUX_FUSION_ETC_DIR)/40-fusion.rules
Ah yes, that's better. Will do for v3.
Thanks!
>
> Danomi -
>
>
> > endef
> >
> > $(eval $(generic-package))
> > --
> > 2.1.1
> >
> > _______________________________________________
> > buildroot mailing list
> > buildroot at busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
--
Guido Martínez, VanguardiaSur
www.vanguardiasur.com.ar
More information about the buildroot
mailing list