From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Jones Subject: Re: [kvm-unit-tests] powerpc: fix boot_rom.bin build Date: Fri, 4 Mar 2016 16:28:50 +0100 Message-ID: <20160304152850.GA1314@localhost.redhat.com> References: <1457038529-4577-1-git-send-email-pfeiner@google.com> <20160304103152.GA3304@localhost.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kvm@vger.kernel.org, pbonzini@redhat.com To: Peter Feiner Return-path: Received: from mx1.redhat.com ([209.132.183.28]:48830 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754763AbcCDP26 (ORCPT ); Fri, 4 Mar 2016 10:28:58 -0500 Content-Disposition: inline In-Reply-To: <20160304103152.GA3304@localhost.redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Fri, Mar 04, 2016 at 11:31:53AM +0100, Andrew Jones wrote: > On Thu, Mar 03, 2016 at 12:55:29PM -0800, Peter Feiner wrote: > > My powerpc objcopy (version 2.23.52.20130305) did a seek() that > > bash process substitution couldn't handle! I've changed the > > Makefile to do things the old fashioned way. > > > > Signed-off-by: Peter Feiner > > --- > > powerpc/Makefile.common | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/powerpc/Makefile.common b/powerpc/Makefile.common > > index 424983e..7818c48 100644 > > --- a/powerpc/Makefile.common > > +++ b/powerpc/Makefile.common > > @@ -48,7 +48,8 @@ FLATLIBS = $(libcflat) $(LIBFDT_archive) > > > > $(TEST_DIR)/boot_rom.bin: $(TEST_DIR)/boot_rom.elf > > dd if=/dev/zero of=$@ bs=256 count=1 > > - $(OBJCOPY) -O binary $^ >(cat - >>$@) > > + $(OBJCOPY) -O binary $^ $@.suffix > > + cat $@.suffix >> $@ > > > > $(TEST_DIR)/boot_rom.elf: CFLAGS = -mbig-endian > > $(TEST_DIR)/boot_rom.elf: $(TEST_DIR)/boot_rom.o > > -- > > 2.7.0.rc3.207.g0ac5344 > > Reviewed-by: Andrew Jones Actually, Unreviewed-by: drew This leaves boot_rom.bin.suffix laying around. We need rm -f $@.suffix too. And, why '.suffix'? How about '.tmp', or even use a tempfile? Thanks, drew > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html