From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [BUG?] rename patch accepted with --dry-run, rejected without (Re: [PATCH V3] arm & sh: factorised duplicated clkdev.c) Date: Fri, 3 Sep 2010 20:29:07 +0100 Message-ID: <20100903192907.GA2978@n2100.arm.linux.org.uk> References: <1283431716-21540-1-git-send-email-plagnioj@jcrosoft.com> <1283434786-26479-1-git-send-email-plagnioj@jcrosoft.com> <20100903182323.GA17152@pengutronix.de> <20100903184351.GC2341@burratino> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , linux-sh@vger.kernel.org, bug-patch@gnu.org, Magnus Damm , linux-arm-kernel@lists.infradead.org, Jean-Christophe PLAGNIOL-VILLARD , git@vger.kernel.org To: Jonathan Nieder X-From: linux-sh-owner@vger.kernel.org Fri Sep 03 21:29:26 2010 Return-path: Envelope-to: glps-linuxsh-dev@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Orbwz-0004DN-E8 for glps-linuxsh-dev@lo.gmane.org; Fri, 03 Sep 2010 21:29:25 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753983Ab0ICT3Y convert rfc822-to-quoted-printable (ORCPT ); Fri, 3 Sep 2010 15:29:24 -0400 Received: from caramon.arm.linux.org.uk ([78.32.30.218]:42177 "EHLO caramon.arm.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753967Ab0ICT3Y (ORCPT ); Fri, 3 Sep 2010 15:29:24 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arm.linux.org.uk; s=caramon; h=Date:From:To:Cc:Subject: Message-ID:References:MIME-Version:Content-Type: Content-Transfer-Encoding:In-Reply-To:Sender; bh=oLY6xV592cDvZuY +4ZDYyyQngPBGxZv+8Bqchf17V00=; b=ovVwlatKyeEZN1bBpxoz2qk1shOEptQ WnnfupdnOuphHFRZtsPhmVUvnCIkMIlBJAP75Sqg+UxsoDXWxa6EzVlHP4Waioo7 3Mqy0/d3X8enU0bE8iztayBbxs92l4033nbh7jOXDellwC9psKZdYkRtognhEQ/2 QxpCGwQmejnI= Received: from n2100.arm.linux.org.uk ([2002:4e20:1eda:1:214:fdff:fe10:4f86]) by caramon.arm.linux.org.uk with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.69) (envelope-from ) id 1Orbwl-0006mF-0X; Fri, 03 Sep 2010 20:29:11 +0100 Received: from linux by n2100.arm.linux.org.uk with local (Exim 4.69) (envelope-from ) id 1Orbwi-0000pP-K3; Fri, 03 Sep 2010 20:29:08 +0100 Content-Disposition: inline In-Reply-To: <20100903184351.GC2341@burratino> User-Agent: Mutt/1.5.19 (2009-01-05) Sender: linux-sh-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-sh@vger.kernel.org Archived-At: On Fri, Sep 03, 2010 at 01:43:51PM -0500, Jonathan Nieder wrote: > (+cc: bug-patch) >=20 > Hi, >=20 > Uwe Kleine-K=F6nig wrote: > > On Fri, Sep 03, 2010 at 07:18:43PM +0900, Magnus Damm wrote: >=20 > >> Using --dry-run is fine, but omitting dry-run gives me: > >>=20 > >> ... > >> patching file arch/arm/common/clkdev.c > >> patching file arch/sh/include/asm/clkdev.h > >> Hunk #1 FAILED at 1. > >> Hunk #2 FAILED at 11. > >> 2 out of 2 hunks FAILED -- saving rejects to file > >> arch/sh/include/asm/clkdev.h.rej > >>=20 > >> I guess this is caused by the last "renaming" hunk, see below. >=20 > Yep, I can reproduce this. Patch applies with "git apply", > "patch --dry-run -p1" accepts it, "patch -p1" fails. git patches include additional metadata for renaming files, which gnu p= atch will not understand. If you want GNU patch compatible diffs, don't use -C or -M when generat= ing patches out of git.