From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guido =?iso-8859-1?Q?Mart=EDnez?= Date: Tue, 18 Nov 2014 16:40:06 -0300 Subject: [Buildroot] [PATCH v3 5/5] package: linux-fusion: use install instead of cp In-Reply-To: <546B9CBF.80905@mind.be> References: <1416244750-24256-1-git-send-email-guido@vanguardiasur.com.ar> <1416244750-24256-6-git-send-email-guido@vanguardiasur.com.ar> <546A714D.3040402@mind.be> <20141118171709.GD20682@fox> <546B9CBF.80905@mind.be> Message-ID: <20141118193804.GA8821@fox> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net On Tue, Nov 18, 2014 at 08:23:43PM +0100, Arnout Vandecappelle wrote: > On 18/11/14 18:17, Guido Mart?nez wrote: > > Hi Arnout, all > > > > On Mon, Nov 17, 2014 at 11:06:05PM +0100, Arnout Vandecappelle wrote: > >> On 17/11/14 18:19, Guido Mart?nez wrote: > >>> in order to not depend on the previous permissions of the file > >>> > >>> Signed-off-by: Guido Mart?nez > >> > >> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) > >> > >> There are a few more suspicious cp instances: > >> > >> board/boundarydevices/nitrogen6x/post-build.sh > >> boot/grub/grub.mk > >> fs/iso9660/iso9660.mk > > I'll take a look at these, thanks. > > > >> Also, it would be good to document somewhere (e.g. in the intro mail) why the > >> rsync in the toolchain is OK. Or even better, replace it even though it's not > >> really necessary - it just feels more consistent and safe. > > Yes, this sounds good. Maybe use --chmod on the toolchain and add a > > comment there? This way we should be covered in the future if someone > > uses 'rsync -a' from there. > > Actually the toolchain already uses --chmod=Du+w. Yes, but that doesn't have any effect on files, so files under output/staging can have just about any mode, and more so when the toolchain is preinstalled somewhere. Changing it to --chmod=u=rwX,go=rX would eliminate that variablity (again, we're assuming that there are no 'random' exec bits set, but that's reasonable since no files are created with the exec bit set). This way, at any place within BR we could do 'rsync -a $(STAGING_DIR)/... $(TARGET_DIR)/...' and have well-defined modes on the target. That kind of rsync (or any other similar copy) don't exist as of now: eveything is done via 'install'. So I don't have a strong opinion for changing the rsync or not. What do you guys think? Thanks! -- Guido Mart?nez, VanguardiaSur www.vanguardiasur.com.ar