Message ID | 1415332238-31677-6-git-send-email-guido@vanguardiasur.com.ar |
---|---|
State | Superseded |
Headers | show |
Guido On Thu, Nov 6, 2014 at 10:50 PM, Guido Martínez <guido@vanguardiasur.com.ar> wrote: > in order to not depend on the previous permissions of the file > > Signed-off-by: Guido Martínez <guido@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 Danomi - > endef > > $(eval $(generic-package)) > -- > 2.1.1 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
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@vanguardiasur.com.ar> wrote: > > in order to not depend on the previous permissions of the file > > > > Signed-off-by: Guido Martínez <guido@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@busybox.net > > http://lists.busybox.net/mailman/listinfo/buildroot
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) endef $(eval $(generic-package))
in order to not depend on the previous permissions of the file Signed-off-by: Guido Martínez <guido@vanguardiasur.com.ar> --- package/linux-fusion/linux-fusion.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)