From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ralf Baechle Subject: Re: git issue / [PATCH] MIPS: fix invalid symbolic link file Date: Thu, 19 Sep 2013 15:39:20 +0200 Message-ID: <20130919133920.GA22468@linux-mips.org> References: <1379596148-32520-1-git-send-email-maddy@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: steven.hill@imgtec.com, mmarek@suse.cz, swarren@nvidia.com, linux-mips@linux-mips.org, linux-kbuild@vger.kernel.org, james.hogan@imgtec.com To: Madhavan Srinivasan , Grant Likely , Rob Herring , devicetree@vger.kernel.org, git@vger.kernel.org X-From: git-owner@vger.kernel.org Thu Sep 19 15:39:42 2013 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1VMeSM-0000oA-3W for gcvg-git-2@plane.gmane.org; Thu, 19 Sep 2013 15:39:42 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752050Ab3ISNji (ORCPT ); Thu, 19 Sep 2013 09:39:38 -0400 Received: from eddie.linux-mips.org ([78.24.191.182]:47910 "EHLO cvs.linux-mips.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751751Ab3ISNjg (ORCPT ); Thu, 19 Sep 2013 09:39:36 -0400 Received: from localhost.localdomain ([127.0.0.1]:55716 "EHLO linux-mips.org" rhost-flags-OK-OK-OK-FAIL) by eddie.linux-mips.org with ESMTP id S6827391Ab3ISNjfjs81J (ORCPT + 2 others); Thu, 19 Sep 2013 15:39:35 +0200 Received: from scotty.linux-mips.net (localhost.localdomain [127.0.0.1]) by scotty.linux-mips.net (8.14.7/8.14.4) with ESMTP id r8JDdQCu026209; Thu, 19 Sep 2013 15:39:26 +0200 Received: (from ralf@localhost) by scotty.linux-mips.net (8.14.7/8.14.7/Submit) id r8JDdKaW026208; Thu, 19 Sep 2013 15:39:20 +0200 Content-Disposition: inline In-Reply-To: <1379596148-32520-1-git-send-email-maddy@linux.vnet.ibm.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: On Thu, Sep 19, 2013 at 06:39:08PM +0530, Madhavan Srinivasan wrote: (Git folks, please read on.) > Commit 3b29aa5ba204c created a symlink file in include/dt-bindings. > Even though commit diff is fine, symlink is invalid. > ls -lb shows a newline character at the end of the filename. > > lrwxrwxrwx 1 maddy maddy 35 Sep 19 18:11 dt-bindings -> > ../../../../../include/dt-bindings\n > > Signed-off-by: Madhavan Srinivasan > --- > arch/mips/boot/dts/include/dt-bindings | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/mips/boot/dts/include/dt-bindings b/arch/mips/boot/dts/include/dt-bindings > index 68ae388..08c00e4 120000 > --- a/arch/mips/boot/dts/include/dt-bindings > +++ b/arch/mips/boot/dts/include/dt-bindings > @@ -1 +1 @@ > -../../../../../include/dt-bindings > +../../../../../include/dt-bindings > \ No newline at end of file > -- > 1.7.10.4 I applied your patch - but now git-show shows it as an empty commit and ls -lb arch/mips/boot/dts/include/dt-bindings still shows the \n at the end of the link target. Things are looking ok now that I manually fixed the link and commited the result. I hope git-push and git-pull are going to handle this correct. So, I wonder if this is a git bug. The original patch that introduced the symlink with the \n is kernel commit 3b29aa5ba204c62b3ec8f9f5b1ebd6e5d74f75d3 and is archived in patchwork at http://patchwork.linux-mips.org/patch/5745/ The patch file contains a \n at the end - but one would expect that from a patch file that has been transfered via email, so I'm not sure how this is supposed to work with emailed patches?!? Anyway, I'm not too fond of sylinks in the tree or in patches and I'm wondering if we could get rid of them for something more bullet proof. Ralf